From 1ddd71f7f0b99c9a116fca618d4c7412ed38e4f8 Mon Sep 17 00:00:00 2001
From: jbjohnso <jbjohnso@8638fb3e-16cb-4fca-ae20-7b5d299a9bcd>
Date: Mon, 23 Nov 2009 18:57:20 +0000
Subject: [PATCH] -Sanitize responses from plugins if they throw non-printable
 characters that are invalid in XML parsing

git-svn-id: https://svn.code.sf.net/p/xcat/code/xcat-core/trunk@4655 8638fb3e-16cb-4fca-ae20-7b5d299a9bcd
---
 xCAT-server/sbin/xcatd | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/xCAT-server/sbin/xcatd b/xCAT-server/sbin/xcatd
index 3b4b02927..9c621f42a 100755
--- a/xCAT-server/sbin/xcatd
+++ b/xCAT-server/sbin/xcatd
@@ -1198,7 +1198,10 @@ sub convey_response {
     build_response($resp);
     return;
   }
-  print $parent_fd XMLout($resp,KeyAttr=>[], NoAttr=>1,RootName=>'xcatresponse');
+  $resp = XMLout($resp,KeyAttr=>[], NoAttr=>1,RootName=>'xcatresponse');
+  #sanitize the response, to avoid being killed by non-printable bytes
+  $resp =~ tr/\011-\177/?/c;
+  print $parent_fd $resp;
   yield; #parent must get timeslice anyway before an ack could possibly return
   my $parsel = new IO::Select;
   $parsel->add($parent_fd);