diff options
| author | Chuck Lever <chuck.lever@oracle.com> | 2009-07-14 16:58:38 -0400 |
|---|---|---|
| committer | Steve Dickson <steved@redhat.com> | 2009-07-14 16:58:38 -0400 |
| commit | ae548c2d55e442c96ad51e90c1e00ce3eb0b718b (patch) | |
| tree | 2b18362ab964cd3a974f8479277c68a00d6960d9 /utils | |
| parent | 6e3fa0e103a494a37cf3a4f9199516923928ab31 (diff) | |
| download | nfs-utils-ae548c2d55e442c96ad51e90c1e00ce3eb0b718b.tar.gz nfs-utils-ae548c2d55e442c96ad51e90c1e00ce3eb0b718b.tar.xz nfs-utils-ae548c2d55e442c96ad51e90c1e00ce3eb0b718b.zip | |
mount.nfs: Fix compiler warning in stropts.c
Address compiler warning:
stropts.c: In function ¿nfs_append_generic_address_option¿:
stropts.c:138: warning: comparison between signed and unsigned
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
Diffstat (limited to 'utils')
| -rw-r--r-- | utils/mount/stropts.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/utils/mount/stropts.c b/utils/mount/stropts.c index e277b3d..9d82bb1 100644 --- a/utils/mount/stropts.c +++ b/utils/mount/stropts.c @@ -130,12 +130,14 @@ static int nfs_append_generic_address_option(const struct sockaddr *sap, { char address[NI_MAXHOST]; char new_option[512]; + int len; if (!nfs_present_sockaddr(sap, salen, address, sizeof(address))) goto out_err; - if (snprintf(new_option, sizeof(new_option), "%s=%s", - keyword, address) >= sizeof(new_option)) + len = snprintf(new_option, sizeof(new_option), "%s=%s", + keyword, address); + if (len < 0 || (size_t)len >= sizeof(new_option)) goto out_err; if (po_append(options, new_option) != PO_SUCCEEDED) |