diff options
| author | Steve Dickson <steved@redhat.com> | 2009-03-23 17:13:01 -0400 |
|---|---|---|
| committer | Steve Dickson <steved@redhat.com> | 2009-03-23 17:13:01 -0400 |
| commit | 3724317e223d46908aac2405bbd73ea2de4f36e5 (patch) | |
| tree | 4498459f0dd3f88349c5126792f3715200397eed /utils/idmapd | |
| parent | d62365079f711b25e73522b2af380abc2a7e2788 (diff) | |
| download | nfs-utils-3724317e223d46908aac2405bbd73ea2de4f36e5.tar.gz nfs-utils-3724317e223d46908aac2405bbd73ea2de4f36e5.tar.xz nfs-utils-3724317e223d46908aac2405bbd73ea2de4f36e5.zip | |
In recent Fedora builds, the '-D _FORTIFY_SOURCE=2' compile
flag has been set. This cause warnings to be generated when
return values from reads/writes (and other calls) are not
checked. The patch address those warnings.
Signed-off-by: Steve Dickson <steved@redhat.com>
Diffstat (limited to 'utils/idmapd')
| -rw-r--r-- | utils/idmapd/idmapd.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/utils/idmapd/idmapd.c b/utils/idmapd/idmapd.c index c1cf4eb..b690e21 100644 --- a/utils/idmapd/idmapd.c +++ b/utils/idmapd/idmapd.c @@ -169,7 +169,10 @@ flush_nfsd_cache(char *path, time_t now) fd = open(path, O_RDWR); if (fd == -1) return -1; - write(fd, stime, strlen(stime)); + if (write(fd, stime, strlen(stime)) != strlen(stime)) { + errx(1, "Flushing nfsd cache failed: errno %d (%s)", + errno, strerror(errno)); + } close(fd); return 0; } @@ -988,7 +991,10 @@ release_parent(void) int status; if (pipefds[1] > 0) { - write(pipefds[1], &status, 1); + if (write(pipefds[1], &status, 1) != 1) { + err(1, "Writing to parent pipe failed: errno %d (%s)\n", + errno, strerror(errno)); + } close(pipefds[1]); pipefds[1] = -1; } |
