From 470a0d14711ee8b807367dc20fcc2c25daba8b87 Mon Sep 17 00:00:00 2001
From: Riccardo Pittau <elfosardo@gmail.com>
Date: Thu, 7 Apr 2022 09:39:56 +0200
Subject: [PATCH] Fix remseqnumber not initialized

The sequence number is never initialized unless the login already
happened, so we need to check for its existence when retrying failed
login.

Change-Id: Ia2319d4c0255d6ae8616cf0c736aaa30d65e6129
---
 pyghmi/ipmi/private/session.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pyghmi/ipmi/private/session.py b/pyghmi/ipmi/private/session.py
index 0da50b9f..8d2fc4aa 100644
--- a/pyghmi/ipmi/private/session.py
+++ b/pyghmi/ipmi/private/session.py
@@ -1348,7 +1348,7 @@ class Session(object):
                 iserver.pktqueue.append(qent)
                 iserver.process_pktqueue()
                 return
-            if (self.remseqnumber is not None
+            if (hasattr(self, 'remseqnumber')
                     and remseqnumber < self.remseqnumber):
                 return -5  # remote sequence number is too low, reject it
             self.remseqnumber = remseqnumber
@@ -1426,7 +1426,7 @@ class Session(object):
             if sid != self.localsid:  # session id mismatch, drop it
                 return
             remseqnumber = struct.unpack("<I", bytes(data[10:14]))[0]
-            if (self.remseqnumber is not None
+            if (hasattr(self, 'remseqnumber')
                     and (remseqnumber < self.remseqnumber)
                     and (self.remseqnumber != 0xffffffff)):
                 return