diff mbox

[4/5] Sanitizing strings before passing them to the logging system.

Message ID 1243019807-21078-4-git-send-email-lmr@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Lucas Meneghel Rodrigues May 22, 2009, 7:16 p.m. UTC
The logging system encodes messages using the utf-8 encoding by
default. So sometimes it's necessary to re-encode lines when
non unicode characters are thrown up by the command outputs.

This patch re-encodes lines of text that will be passed to the
logging system on the track_process function on kvm_utils.
---
 client/tests/kvm_runtest_2/kvm_utils.py |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)
diff mbox

Patch

diff --git a/client/tests/kvm_runtest_2/kvm_utils.py b/client/tests/kvm_runtest_2/kvm_utils.py
index bcc80a1..9ef5954 100644
--- a/client/tests/kvm_runtest_2/kvm_utils.py
+++ b/client/tests/kvm_runtest_2/kvm_utils.py
@@ -755,6 +755,9 @@  def track_process(sub, status_output=None, term_func=None, stdout_func=None,
         # Call stdout_func with the returned text
         if stdout_func:
             text = prefix + text.strip()
+            # We need to sanitize the text before passing it to the logging
+            # system
+            text = text.decode('utf-8', 'replace')
             stdout_func(text)