Menu

#67 TCPIPCancelDNR doesn't work

3.0b8
pending-fixed
5
2014-08-27
2013-08-15
No

TCPIPCancelDNR doesn't work. It's comparing the wrong thing.

CMP 4+2+2+RAWSTACKEDPARMS,S
CMP 4+2+2+RAWSTACKEDPARMS+2,S

should be

CMP 4+2+2+1+RAWSTACKEDPARMS,S
CMP 4+2+2+1+RAWSTACKEDPARMS+2,S

to account for the PHB that happened earlier.

LDY #DNRbuffptr
LDA [TMPTR],Y
STA TMPTR1
CMP 4+2+2+RAWSTACKEDPARMS,S
BNE :NEXT
INY
INY
LDA [TMPTR],Y
STA TMPTR1+2
CMP 4+2+2+RAWSTACKEDPARMS+2,S
BNE :NEXT

Discussion

  • Andrew Roughan

    Andrew Roughan - 2013-08-15

    Kelvin patched it in memory and verified (capture packet, monitoring the DNS queue, and regular testing) that TCPIPDNRNameToIP works as expected.

    This fix solves the problems reported in issue #66

     

    Last edit: Andrew Roughan 2013-08-16
  • Andrew Roughan

    Andrew Roughan - 2014-08-27

    Source code changed on Sweet16:testbuild:Marinetti:Init:I.DNR.S
    Changelog.b8
    Released to testers in 3.0b8.2

     

    Last edit: Andrew Roughan 2014-08-28

Log in to post a comment.

MongoDB Logo MongoDB