From ba699294ca7d902f95ed97fdfd3bcc8d2587740e Mon Sep 17 00:00:00 2001 From: Connor Reed Date: Tue, 25 Feb 2020 11:20:46 -0500 Subject: [PATCH] fix(kcs): Fix the command class to use bytearray instead of lists for raw data Change-Id: I55ef076827447d3e0b8bbbd9d9dd8d0006804232 --- pyghmi/ipmi/command.py | 2 -- pyghmi/ipmi/private/localsession.py | 4 ++-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/pyghmi/ipmi/command.py b/pyghmi/ipmi/command.py index c6225ee3..00fe7460 100644 --- a/pyghmi/ipmi/command.py +++ b/pyghmi/ipmi/command.py @@ -495,8 +495,6 @@ class Command(object): bridge_request=bridge_request, data=data, delay_xmit=delay_xmit, retry=retry, timeout=timeout) - if 'data' in rsp: - rsp['data'] = list(rsp['data']) return rsp def get_power(self): diff --git a/pyghmi/ipmi/private/localsession.py b/pyghmi/ipmi/private/localsession.py index d1983efd..7e23c4a8 100644 --- a/pyghmi/ipmi/private/localsession.py +++ b/pyghmi/ipmi/private/localsession.py @@ -98,8 +98,8 @@ class Session(object): def parsed_rsp(self): response = {'netfn': self.rsp.msg.netfn, 'command': self.rsp.msg.cmd, 'code': bytearray(self.databuffer.raw)[0], - 'data': list(bytearray( - self.databuffer.raw[1:self.rsp.msg.data_len]))} + 'data': bytearray( + self.databuffer.raw[1:self.rsp.msg.data_len])} errorstr = iutil.get_ipmi_error(response) if errorstr: response['error'] = errorstr