server/chal.c: Don't load sequence number twice.

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.

diff --git a/server/chal.c b/server/chal.c
index e282f08548967e5cd8d4cd7722948a0cb5e4c01f..387f0db1fb29e182d36b28d9df979d7ebce21082 100644 (file)
--- a/server/chal.c
+++ b/server/chal.c
@@ -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);