diff options
| author | Arnold D. Robbins <arnold@skeeve.com> | 2014-11-23 19:56:27 +0200 |
|---|---|---|
| committer | Arnold D. Robbins <arnold@skeeve.com> | 2014-11-23 19:56:27 +0200 |
| commit | 2513062a4c89b0b60c3717d506fce841d44d871e (patch) | |
| tree | ae2a21e0a0e0173c1c8b44e9e087da1ab357d66b /extension | |
| parent | 838f65088cda84edc2df609d3e388acb3c8eb13d (diff) | |
| download | egawk-2513062a4c89b0b60c3717d506fce841d44d871e.tar.gz egawk-2513062a4c89b0b60c3717d506fce841d44d871e.tar.bz2 egawk-2513062a4c89b0b60c3717d506fce841d44d871e.zip | |
Fix compile problems and warnings on current GCC (4.9.2).
Diffstat (limited to 'extension')
| -rw-r--r-- | extension/ChangeLog | 5 | ||||
| -rw-r--r-- | extension/inplace.c | 8 |
2 files changed, 11 insertions, 2 deletions
diff --git a/extension/ChangeLog b/extension/ChangeLog index 940f7f15..41c8a0e4 100644 --- a/extension/ChangeLog +++ b/extension/ChangeLog @@ -1,3 +1,8 @@ +2014-11-23 Arnold D. Robbins <arnold@skeeve.com> + + * inplace.c (do_inplace_begin): Jump through hoops to silence + GCC warnings about return value of chown. + 2014-10-12 Arnold D. Robbins <arnold@skeeve.com> * Makefile.am (uninstall-so): Remove *.lib too, per suggestion diff --git a/extension/inplace.c b/extension/inplace.c index 8a7375c4..0693ad92 100644 --- a/extension/inplace.c +++ b/extension/inplace.c @@ -170,8 +170,12 @@ do_inplace_begin(int nargs, awk_value_t *result) state.tname, strerror(errno)); /* N.B. chown/chmod should be more portable than fchown/fchmod */ - if (chown(state.tname, sbuf.st_uid, sbuf.st_gid) < 0) - (void) chown(state.tname, -1, sbuf.st_gid); + if (chown(state.tname, sbuf.st_uid, sbuf.st_gid) < 0) { + /* jumping through hoops to silence gcc. :-( */ + int junk; + junk = chown(state.tname, -1, sbuf.st_gid); + junk = junk; + } if (chmod(state.tname, sbuf.st_mode) < 0) fatal(ext_id, _("inplace_begin: chmod failed (%s)"), |