Fix leak in forward-search
Valgrind reports the below leak. Fix the leak by using xrealloc, even for the first allocation, as buf is static. ==29158== 5,888 bytes in 23 blocks are definitely lost in loss record 3,028 of 3,149 ==29158== at 0x4C2BE2D: malloc (vg_replace_malloc.c:299) ==29158== by 0x41B557: xmalloc (common-utils.c:44) ==29158== by 0x60B7D9: forward_search_command(char const*, int) (source.c:1563) ==29158== by 0x40BA68: cmd_func(cmd_list_element*, char const*, int) (cli-decode.c:1888) ==29158== by 0x665300: execute_command(char const*, int) (top.c:630) ... gdb/ChangeLog 2018-11-29 Philippe Waroquiers <philippe.waroquiers@skynet.be> * source.c (forward_search_command): Fix leak by using xrealloc even for the first allocation in the loop, as buf is static.
This commit is contained in:
parent
e61667ef14
commit
95b1f9ac6b
|
@ -1,3 +1,9 @@
|
|||
2018-11-29 Philippe Waroquiers <philippe.waroquiers@skynet.be>
|
||||
|
||||
* source.c (forward_search_command): Fix leak by using
|
||||
xrealloc even for the first allocation in the loop, as buf
|
||||
is static.
|
||||
|
||||
2018-11-29 Rajendra SY <rajendra.sy@gmail.com>
|
||||
|
||||
PR gdb/23093
|
||||
|
|
|
@ -1560,7 +1560,7 @@ forward_search_command (const char *regex, int from_tty)
|
|||
int cursize, newsize;
|
||||
|
||||
cursize = 256;
|
||||
buf = (char *) xmalloc (cursize);
|
||||
buf = (char *) xrealloc (buf, cursize);
|
||||
p = buf;
|
||||
|
||||
c = fgetc (stream.get ());
|
||||
|
|
Loading…
Reference in New Issue