diff options
| author | Brian Behlendorf <[email protected]> | 2011-03-18 13:54:27 -0700 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2011-03-18 13:54:49 -0700 |
| commit | ec49a5f0ec020fa1b0d3e915e2ffb10bbed97599 (patch) | |
| tree | 9a55659688060e6fd46e5a6e4c1ddeb1cf61e795 /cmd | |
| parent | 04516a45b27144cf42bec005ef2de6d505d236a5 (diff) | |
Fix getcwd() warning
New versions glibc declare getcwd() with the warn_unused_result attribute.
This results in a warning because the updated mount helper was not
checking this return value. This issue was fixed by checking the return
type and in the case of an error simply returning the passed dataset.
One possible, but unlikely, error would be having your cwd directory
unlinked while the mount command was running.
cmd/mount_zfs/mount_zfs.c: In function ‘parse_dataset’:
cmd/mount_zfs/mount_zfs.c:223:2: error: ignoring return value of
‘getcwd’, declared with attribute warn_unused_result
Diffstat (limited to 'cmd')
| -rw-r--r-- | cmd/mount_zfs/mount_zfs.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/cmd/mount_zfs/mount_zfs.c b/cmd/mount_zfs/mount_zfs.c index fbb954acd..090c915ce 100644 --- a/cmd/mount_zfs/mount_zfs.c +++ b/cmd/mount_zfs/mount_zfs.c @@ -220,7 +220,9 @@ parse_dataset(char *dataset) char cwd[PATH_MAX]; int len; - (void) getcwd(cwd, PATH_MAX); + if (getcwd(cwd, PATH_MAX) == NULL) + return (dataset); + len = strlen(cwd); /* Do not add one when cwd already ends in a trailing '/' */ |