diff options
| author | Nikola Pajkovsky <npajkovs@redhat.com> | 2011-03-30 14:01:52 +0200 |
|---|---|---|
| committer | Nikola Pajkovsky <npajkovs@redhat.com> | 2011-03-30 15:14:25 +0200 |
| commit | 19958ce67a9870b32a5e99df4a9d6f7e53759bcb (patch) | |
| tree | ffac166a82d4c734dd8b18855435675e465a00e4 /src/lib | |
| parent | b21745640ee0852ce0cdc3f430371769a60d9317 (diff) | |
| download | abrt-19958ce67a9870b32a5e99df4a9d6f7e53759bcb.tar.gz abrt-19958ce67a9870b32a5e99df4a9d6f7e53759bcb.tar.xz abrt-19958ce67a9870b32a5e99df4a9d6f7e53759bcb.zip | |
better validation for numbers
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
Diffstat (limited to 'src/lib')
| -rw-r--r-- | src/lib/event_config.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/lib/event_config.c b/src/lib/event_config.c index 7a8b5e31..6b8a288c 100644 --- a/src/lib/event_config.c +++ b/src/lib/event_config.c @@ -304,10 +304,13 @@ static char *validate_event_option(event_option_t *opt) break; case OPTION_TYPE_NUMBER: { - long r = strtol(opt->value, (char **)&s, 10); + char *endptr; + errno = 0; + long r = strtol(opt->value, &endptr, 10); (void) r; - if (*s || errno) + if (errno != 0 || endptr == opt->value || *endptr != '\0') return xasprintf(_("Invalid number '%s'"), opt->value); + break; } case OPTION_TYPE_BOOL: |