patman: Convert byte arrays to strings
authorGeorge McCollister <george.mccollister@gmail.com>
Thu, 30 Mar 2017 14:44:24 +0000 (09:44 -0500)
committerSimon Glass <sjg@chromium.org>
Thu, 13 Apr 2017 17:43:49 +0000 (11:43 -0600)
commit6db06f94e19d539bb7001a666e7c52672e0682ea
tree762cc5ca496a8780ed826bed854fd07fdeb887c5
parent6f8c351e98726ec9b569bf16f7efa2a4e4648727
patman: Convert byte arrays to strings

os.read() returns a byte array in Python 3.5.2 and needs to be converted
into a string. Check if the returned value is an instance of bytes and
if it is decode it as a utf-8 string. If it is not a utf-8 encoded string
the decoding may fail with an exception.

Prior to this fix the comparisions check data == "" would fail when data
was b'' and would cause an infinite memory leaking loop. joins would
also fail with an exception below but due to the infinite loop it never
made it that far.

Signed-off-by: George McCollister <george.mccollister@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>
tools/patman/cros_subprocess.py