lgandx/Responder

RunFinger.py: Signing not recognized

massimiliano-dalcero opened this issue · 1 comments

Hello,
I cloned as is the project and I noticed that signing was not recognized correctly: it was always set to false even when it was enabled

With Pyhon 3.10 and 3.11, it works by modifying the code:

FROM:

SMB2SigningMandatory(data):
     global SMB2signing
     if data[70:71] == b"\x03":
         SMB2signing = "True"
     else:
         SMB2signing = "False"

TO:

SMB2SigningMandatory(data):
     global SMB2signing
     if data[70:71] == "\x03":
         SMB2signing = "True"
     else:
         SMB2signing = "False"

However, I don't know if the byte prefix problem is wider and needs to be modified elsewhere too.

I replaced all "b" prefix except to:

def ParseNegotiateSMB2Ans(data):
    if data[4:8] == b"\xfeSMB":
        return True
    else:
        return False

And now, appear to works, but I don't know the global side effects.

Good catch, this has been fixed in 413bc8b

Cheers,