Fix handling command line option --xml-socket.

Fixes BZ#382998
Patch by: Orgad Shaneh <orgads@gmail.com>

git-svn-id: svn://svn.valgrind.org/valgrind/trunk@16467

diff --git a/NEWS b/NEWS
index 9b39447cab878ca1d7cde69909fcc17c535d3f4f..bb60b81abe7d34e3ed910507c87968da2cea0b5e 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -45,6 +45,7 @@ where XXXXXX is the bug number as listed below.
 382256  gz compiler flag test doesn't work for gold
 382407  vg_perf needs "--terse" command line option
 382515  "Assertion 'di->have_dinfo' failed." on wine's dlls/mscoree/tests/mscoree.c
+382998  xml-socket doesn't work
 
 Release 3.13.0 (15 June 2017)
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

diff --git a/coregrind/m_libcprint.c b/coregrind/m_libcprint.c
index d66c67dc6488dedec920053a7c5a2b1066300f97..f6ba2022e278e2bf50df9495461e96902d8ee99a 100644 (file)
--- a/coregrind/m_libcprint.c
+++ b/coregrind/m_libcprint.c
@@ -526,7 +526,7 @@ void VG_(init_log_xml_sinks)(VgLogTo log_to, VgLogTo xml_to,
          break;
 
       case VgLogTo_Socket:
-         log_fd = prepare_sink_socket(VG_(clo_xml_fname_unexpanded),
+         xml_fd = prepare_sink_socket(VG_(clo_xml_fname_unexpanded),
                                       &VG_(xml_output_sink), True);
          break;
    }