]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
Get rid of Werror=maybe-uninitialized in res_send.c.
authorStefan Liebler <stli@linux.ibm.com>
Mon, 20 Jan 2020 09:55:55 +0000 (10:55 +0100)
committerStefan Liebler <stli@linux.ibm.com>
Mon, 20 Jan 2020 09:57:08 +0000 (10:57 +0100)
The commit 446997ff1433d33452b81dfa9e626b8dccf101a4 introduced
this new usage of resplen. If build with gcc 9 -march>=z13 on s390x,
the following warning occurs:
res_send.c: In function ‘__res_context_send’:
res_send.c:539:6: error: ‘resplen’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
  539 |   if (resplen > HFIXEDSZ)
      |      ^

Therefore this patch adds a further DIAG_IGNORE_NEEDS_COMMENT in the
same way as it was previously done for usages of resplen or n.
See commit d1bc2cbbed9aea2017ef941f63c8786571da5b4f.

resolv/res_send.c

index 89c40ce013733536c2f901950d70ab9425462d50..7e5fec66466e16f88fe950baed55b371721abac0 100644 (file)
@@ -534,6 +534,9 @@ __res_context_send (struct resolv_context *ctx,
 
                resplen = n;
 
+               /* See comment at the declaration of n.  Note: resplen = n;  */
+               DIAG_PUSH_NEEDS_COMMENT;
+               DIAG_IGNORE_NEEDS_COMMENT (9, "-Wmaybe-uninitialized");
                /* Mask the AD bit in both responses unless it is
                   marked trusted.  */
                if (resplen > HFIXEDSZ)
@@ -543,6 +546,7 @@ __res_context_send (struct resolv_context *ctx,
                    else
                      mask_ad_bit (ctx, ans);
                  }
+               DIAG_POP_NEEDS_COMMENT;
                if (resplen2 != NULL && *resplen2 > HFIXEDSZ)
                  mask_ad_bit (ctx, *ansp2);