diff options
| author | John Bowler <jbowler@nslu2-linux.org> | 2006-02-01 02:05:17 +0000 |
|---|---|---|
| committer | OpenEmbedded Project <openembedded-devel@lists.openembedded.org> | 2006-02-01 02:05:17 +0000 |
| commit | b0960543c749e1812e685ae0a9865bdc2d3896ae (patch) | |
| tree | 81e410c21b8d3b0bd934fec290b28ef7d52e8cfa /packages/binutils/binutils-2.16 | |
| parent | a0dbc5fc54bd136f2c34ed3f82e84b8f84726570 (diff) | |
binutils: make strip and objcopy return 0 if asked to write ro files in 2.16
- the same bug exists in earlier versions and is probably fixed by the
same patch, objcopy (==strip) would identify an ro file and abort the
overwrite (smart_rename), but the caller in objcopy fails to check the
error code and so the error is not reported
Diffstat (limited to 'packages/binutils/binutils-2.16')
| -rw-r--r-- | packages/binutils/binutils-2.16/binutils-2.16-objcopy-rename-errorcode.patch | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/packages/binutils/binutils-2.16/binutils-2.16-objcopy-rename-errorcode.patch b/packages/binutils/binutils-2.16/binutils-2.16-objcopy-rename-errorcode.patch new file mode 100644 index 0000000000..e4372de88f --- /dev/null +++ b/packages/binutils/binutils-2.16/binutils-2.16-objcopy-rename-errorcode.patch @@ -0,0 +1,29 @@ +# strip (and objcopy) fail to set the error code if there is no +# output file name and the rename of the stripped (or copied) file +# fails, yet the command fails to do anything. This fixes both +# objcopy and strip. +# +#Signed-off-by: John Bowler <jbowler@acm.org> + +--- binutils-2.16/binutils/objcopy.c.orig 2006-01-31 11:15:38.797318519 -0800 ++++ binutils-2.16/binutils/objcopy.c 2006-01-31 11:15:40.463318516 -0800 +@@ -2434,7 +2434,8 @@ strip_main (int argc, char *argv[]) + if (preserve_dates) + set_times (tmpname, &statbuf); + if (output_file == NULL) +- smart_rename (tmpname, argv[i], preserve_dates); ++ if (smart_rename (tmpname, argv[i], preserve_dates)) ++ hold_status = 1; + status = hold_status; + } + else +@@ -3013,7 +3014,8 @@ copy_main (int argc, char *argv[]) + { + if (preserve_dates) + set_times (tmpname, &statbuf); +- smart_rename (tmpname, input_filename, preserve_dates); ++ if (smart_rename (tmpname, input_filename, preserve_dates)) ++ status = 1; + } + else + unlink (tmpname); |