diff options
| author | Stefan Liebler <stli@linux.ibm.com> | 2020-01-20 10:55:55 +0100 |
|---|---|---|
| committer | Stefan Liebler <stli@linux.ibm.com> | 2020-01-20 10:57:08 +0100 |
| commit | aba12794dc0d1835b9652f34df722970b1bd3397 (patch) | |
| tree | 7cc58cd7f1286cf2a52cc78d60f74dd5902bb4a1 | |
| parent | 3cb57302ed6acc34409c71cfd99e4f22359d6533 (diff) | |
| download | glibc-aba12794dc0d1835b9652f34df722970b1bd3397.zip glibc-aba12794dc0d1835b9652f34df722970b1bd3397.tar.gz glibc-aba12794dc0d1835b9652f34df722970b1bd3397.tar.bz2 | |
Get rid of Werror=maybe-uninitialized in res_send.c.
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.
| -rw-r--r-- | resolv/res_send.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/resolv/res_send.c b/resolv/res_send.c index 89c40ce..7e5fec6 100644 --- a/resolv/res_send.c +++ b/resolv/res_send.c @@ -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); |