diff options
author | Darren Tucker <dtucker@zip.com.au> | 2007-02-25 20:38:55 +1100 |
---|---|---|
committer | Darren Tucker <dtucker@zip.com.au> | 2007-02-25 20:38:55 +1100 |
commit | 90aaed43979c6b4e42b41ef8dc2a970df248c2c7 (patch) | |
tree | d691df6b05851c7288798046e6d38d9c63b2f6c7 /moduli.c | |
parent | 82347a8fd67d294537b0a95e8ea5b7b7912ced11 (diff) |
- ray@cvs.openbsd.org 2007/02/24 03:30:11
[moduli.c]
- strlen returns size_t, not int.
- Pass full buffer size to fgets.
OK djm@, millert@, and moritz@.
Diffstat (limited to 'moduli.c')
-rw-r--r-- | moduli.c | 8 |
1 files changed, 3 insertions, 5 deletions
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: moduli.c,v 1.19 2006/11/06 21:25:28 markus Exp $ */ | 1 | /* $OpenBSD: moduli.c,v 1.20 2007/02/24 03:30:11 ray Exp $ */ |
2 | /* | 2 | /* |
3 | * Copyright 1994 Phil Karn <karn@qualcomm.com> | 3 | * Copyright 1994 Phil Karn <karn@qualcomm.com> |
4 | * Copyright 1996-1998, 2003 William Allen Simpson <wsimpson@greendragon.com> | 4 | * Copyright 1996-1998, 2003 William Allen Simpson <wsimpson@greendragon.com> |
@@ -490,11 +490,9 @@ prime_test(FILE *in, FILE *out, u_int32_t trials, u_int32_t generator_wanted) | |||
490 | 490 | ||
491 | res = 0; | 491 | res = 0; |
492 | lp = xmalloc(QLINESIZE + 1); | 492 | lp = xmalloc(QLINESIZE + 1); |
493 | while (fgets(lp, QLINESIZE, in) != NULL) { | 493 | while (fgets(lp, QLINESIZE + 1, in) != NULL) { |
494 | int ll = strlen(lp); | ||
495 | |||
496 | count_in++; | 494 | count_in++; |
497 | if (ll < 14 || *lp == '!' || *lp == '#') { | 495 | if (strlen(lp) < 14 || *lp == '!' || *lp == '#') { |
498 | debug2("%10u: comment or short line", count_in); | 496 | debug2("%10u: comment or short line", count_in); |
499 | continue; | 497 | continue; |
500 | } | 498 | } |