diff options
| author | xebd <xeb@mail.ru> | 2021-03-17 11:30:45 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-03-17 11:30:45 +0300 |
| commit | 02ee3cf8abebec16bf9106426e095495144c3d2f (patch) | |
| tree | d871c149812c68b7ceee591f6bb265f4ad643414 /accel-pppd/logs | |
| parent | 49ef6cf969f662c44f4be2b82b101273c8c6de71 (diff) | |
| parent | 2785d81ec848ebb920c8a612b4eae7876a619a18 (diff) | |
| download | accel-ppp-02ee3cf8abebec16bf9106426e095495144c3d2f.tar.gz accel-ppp-02ee3cf8abebec16bf9106426e095495144c3d2f.zip | |
Merge pull request #153 from anphsw/master
Some bugfixes found by cppcheck
Diffstat (limited to 'accel-pppd/logs')
| -rw-r--r-- | accel-pppd/logs/log_syslog.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/accel-pppd/logs/log_syslog.c b/accel-pppd/logs/log_syslog.c index 562d895..f06c1cc 100644 --- a/accel-pppd/logs/log_syslog.c +++ b/accel-pppd/logs/log_syslog.c @@ -138,19 +138,21 @@ static void parse_opt(const char *opt, char **ident, int *facility) int n; const char *facility_name[] = {"daemon", "local0", "local1", "local2", "local3", "local4", "local5", "local6", "local7"}; const int facility_num[] = {LOG_DAEMON, LOG_LOCAL0, LOG_LOCAL1, LOG_LOCAL2, LOG_LOCAL3, LOG_LOCAL4, LOG_LOCAL5, LOG_LOCAL6, LOG_LOCAL7}; + const int facility_total = 9; ptr = strchr(str, ','); if (ptr) { *ptr = 0; n = strtol(ptr + 1, &endptr, 10); if (*endptr) { - for (n = 0; n < sizeof(facility_name); n++) { + for (n = 0; n < facility_total; n++) { if (!strcasecmp(ptr + 1, facility_name[n])) break; } - if (n == sizeof(facility_name)) + if (n == facility_total) { log_emerg("log_syslog: unknown facility name '%s'\n", ptr + 1); - else + *facility = LOG_DAEMON; + } else *facility = facility_num[n]; } else *facility = n; |