|
From: <sv...@va...> - 2014-09-15 20:20:32
|
Author: philippe
Date: Mon Sep 15 20:20:24 2014
New Revision: 14541
Log:
Fix leak that can happen in case of IO error between V gdbsrv and vgdb.
Spotted by Florian Krohm/IBM BEAM checker
Modified:
trunk/coregrind/m_gdbserver/remote-utils.c
Modified: trunk/coregrind/m_gdbserver/remote-utils.c
==============================================================================
--- trunk/coregrind/m_gdbserver/remote-utils.c (original)
+++ trunk/coregrind/m_gdbserver/remote-utils.c Mon Sep 15 20:20:24 2014
@@ -836,6 +836,7 @@
/* we might have to write a pkt when out FIFO not yet/anymore opened */
if (!ensure_write_remote_desc()) {
warning ("putpkt(write) error: no write_remote_desc\n");
+ free (buf2);
return -1;
}
@@ -845,6 +846,7 @@
do {
if (VG_(write) (write_remote_desc, buf2, p - buf2) != p - buf2) {
warning ("putpkt(write) error\n");
+ free (buf2);
return -1;
}
|