Fix: undo untested breaking changes from fortra#1311 that change getE…

…rrorString (fortra#1714) * fix(nterrors): undo untested breaking changes from fortra#1311 that change getErrorString for nt_status errors * fix(nt_errors): had incorrect older reference, updated to nt_errors * Update impacket/krb5/kerberosv5.py Co-authored-by: Gabriel Gonzalez <gabriel.gonzalez@fortra.com> --------- Co-authored-by: Gabriel Gonzalez <gabriel.gonzalez@fortra.com>
Tw1sm · Mar 18, 2024 · 0d2b72a · 0d2b72a
1 parent 4a62f39
commit 0d2b72a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/impacket/krb5/kerberosv5.py b/impacket/krb5/kerberosv5.py
@@ -712,7 +712,7 @@ def getErrorPacket( self ):
         return self.packet
 
     def getErrorString( self ):
-        return str(self)
+        return constants.ERROR_MESSAGES[self.error]
 
     def __str__( self ):
         retString = 'Kerberos SessionError: %s(%s)' % (constants.ERROR_MESSAGES[self.error])

diff --git a/impacket/smbconnection.py b/impacket/smbconnection.py
@@ -986,7 +986,7 @@ def getErrorPacket( self ):
         return self.packet
 
     def getErrorString( self ):
-        return str(self)
+        return nt_errors.ERROR_MESSAGES[self.error]
 
     def __str__( self ):
         key = self.error