chiark / gitweb /
server/chal.c: Don't load sequence number twice.
authorMark Wooding <mdw@distorted.org.uk>
Tue, 11 Mar 2014 14:32:12 +0000 (14:32 +0000)
committerMark Wooding <mdw@distorted.org.uk>
Tue, 11 Mar 2014 15:30:39 +0000 (15:30 +0000)
This is especially embarrassing because the only other reason the
sequence number is loaded is for a trace message.  This was discovered
from compiler warning about `seq' being unused when tracing is disabled.

server/chal.c

index e282f08..387f0db 100644 (file)
@@ -124,7 +124,7 @@ int c_check(buf *b)
     goto fail;
   }
   seq = LOAD32(p);
-  if (seq_check(&iseq, LOAD32(p), "CHAL"))
+  if (seq_check(&iseq, seq, "CHAL"))
     goto fail;
   T( trace(T_CHAL, "chal: checked challenge %lu", (unsigned long)seq); )
   return (0);