diff options
author | Siddhesh Poyarekar <siddhesh@redhat.com> | 2014-07-09 10:36:50 +0530 |
---|---|---|
committer | Siddhesh Poyarekar <siddhesh@redhat.com> | 2014-07-09 10:36:50 +0530 |
commit | 6ee3eab42979fdd5bb971a40fec32e04bb9dd2ad (patch) | |
tree | 70452bbbb53f12492f2685494adca1b079637525 | |
parent | 6680738a18fc219de2e4f7be1d53e4fb251f1166 (diff) | |
download | glibc-6ee3eab42979fdd5bb971a40fec32e04bb9dd2ad.zip glibc-6ee3eab42979fdd5bb971a40fec32e04bb9dd2ad.tar.gz glibc-6ee3eab42979fdd5bb971a40fec32e04bb9dd2ad.tar.bz2 |
Fix -Wmaybe-uninitialized warning in xdr.c
While we're at fixing build warnings, here's one unnecessary warning
that can be fixed fairly easily. The SIZE variable is never actually
use uninitialized, but the compiler cannot make that out and thinks
(correctly) that there is a potential for accessing SIZE without
initializing it. Make this safe by initializing SIZE to 0.
Tested on x86_64.
-rw-r--r-- | ChangeLog | 2 | ||||
-rw-r--r-- | sunrpc/xdr.c | 2 |
2 files changed, 3 insertions, 1 deletions
@@ -1,5 +1,7 @@ 2014-07-09 Siddhesh Poyarekar <siddhesh@redhat.com> + * sunrpc/xdr.c (xdr_string): Initialize SIZE to 0. + * sysdeps/generic/unwind-pe.h: Only check if __cplusplus is defined. diff --git a/sunrpc/xdr.c b/sunrpc/xdr.c index b3f96ca..129abd8 100644 --- a/sunrpc/xdr.c +++ b/sunrpc/xdr.c @@ -739,7 +739,7 @@ xdr_string (xdrs, cpp, maxsize) u_int maxsize; { char *sp = *cpp; /* sp is the actual string pointer */ - u_int size; + u_int size = 0; u_int nodesize; /* |