diff options
| author | Lukas Fleischer <calcurse@cryptocrack.de> | 2012-03-01 23:15:38 +0100 |
|---|---|---|
| committer | Lukas Fleischer <calcurse@cryptocrack.de> | 2012-03-12 20:36:22 +0100 |
| commit | 2c9499bf272e06a62902711c6c20621ef3f80e64 (patch) | |
| tree | 6c71560531acf606557a586fa3cdf712e55b2960 /src/config.c | |
| parent | 7072c9c88a5d54b3f22256e6cf0015e1900374da (diff) | |
| download | calcurse-2c9499bf272e06a62902711c6c20621ef3f80e64.tar.gz calcurse-2c9499bf272e06a62902711c6c20621ef3f80e64.zip | |
Use strcmp() instead of strncmp()
strncmp() isn't intended to be a secure strcmp() replacement, it is
designed to be used if you want to compare the first n characters of two
strings. Since we always compare character pointers with string
literals, switch to using strcmp() everywhere.
Signed-off-by: Lukas Fleischer <calcurse@cryptocrack.de>
Diffstat (limited to 'src/config.c')
| -rw-r--r-- | src/config.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/config.c b/src/config.c index 6355781..acc1584 100644 --- a/src/config.c +++ b/src/config.c @@ -117,9 +117,9 @@ typedef int (*config_fn_walk_junk_cb_t) (const char *, void *); static int config_parse_bool (unsigned *dest, const char *val) { - if (strncmp (val, "yes", 4) == 0) + if (strcmp (val, "yes") == 0) *dest = 1; - else if (strncmp (val, "no", 3) == 0) + else if (strcmp (val, "no") == 0) *dest = 0; else return 0; |