diff options
| author | Denys Vlasenko <vda.linux@googlemail.com> | 2018-07-27 12:14:39 +0200 | 
|---|---|---|
| committer | Denys Vlasenko <vda.linux@googlemail.com> | 2018-07-27 12:14:39 +0200 | 
| commit | 186cf4976768029113cf8438734a65bf2c489c5c (patch) | |
| tree | 0b2c2a4acad5472d65d87d7274cf91da2152d6b4 /scripts/basic/fixdep.c | |
| parent | 7c5f18a3bab721cdfa515220ad8d481643aaae23 (diff) | |
| download | busybox-186cf4976768029113cf8438734a65bf2c489c5c.tar.gz | |
hush: in some cases, expand_on_ifs() relied of uninitialized memory
The n > 0 check to prevent access to the last byte of non-existing argv[-1]
wasn't enough. Switched to making sure there are initialized (zero) bytes there.
A predictable testcase is rather hard to construct, unfortunately,
contents of memory depends on allocator behavior and whatnot.
function                                             old     new   delta
o_save_ptr_helper                                    119     137     +18
expand_on_ifs                                        345     339      -6
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 1/1 up/down: 18/-6)              Total: 12 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'scripts/basic/fixdep.c')
0 files changed, 0 insertions, 0 deletions
