diff options
| author | Tom Tromey <tom@tromey.com> | 2021-12-26 13:49:22 -0700 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2022-01-06 08:37:19 -0700 |
| commit | 5bd1d4c158953c17ad33fdb34c0919f2b32ddc45 (patch) | |
| tree | d35e4f9c4e03f6b90e15ec182c2c2883b3f84971 /libiberty/strcasecmp.c | |
| parent | 5220f61a22a0c1a84403725532d6359aaeb3d2fa (diff) | |
| download | binutils-5bd1d4c158953c17ad33fdb34c0919f2b32ddc45.zip binutils-5bd1d4c158953c17ad33fdb34c0919f2b32ddc45.tar.gz binutils-5bd1d4c158953c17ad33fdb34c0919f2b32ddc45.tar.bz2 | |
Clean up some dead code in windows-tdep.c
windows-tdep.c checks the result of xmalloc, which isn't necessary. I
initially removed this dead check, but then went a bit further and
modified the code so that some "goto"s and explicit memory management
could be removed. Then, I added a couple of missing bounds checks.
I believe this also fixes a possible bug with a missing 0-termination
of a string. I am not certain, but that is why I think the existing
code allocates a buffer that is 1 byte too long -- but then it fails
to set this byte to 0.
Diffstat (limited to 'libiberty/strcasecmp.c')
0 files changed, 0 insertions, 0 deletions