New Defects reported by Coverity Scan for NetBSD-amd64-user

[scan-admin%coverity.com@localhost]

Hi,

Please find the latest report on new defect(s) introduced to NetBSD-amd64-user found with Coverity Scan.

1 new defect(s) introduced to NetBSD-amd64-user found with Coverity Scan.
8 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan.

New defect(s) Reported-by: Coverity Scan
Showing 1 of 1 defect(s)


** CID 980936:    (USE_AFTER_FREE)
/usr.sbin/syslogd/syslogd.c: 3188 in read_config_file()
/usr.sbin/syslogd/syslogd.c: 3167 in read_config_file()


________________________________________________________________________________________________________
*** CID 980936:    (USE_AFTER_FREE)
/usr.sbin/syslogd/syslogd.c: 3188 in read_config_file()
3182     				if (credhead) do {
3183     					if(!(cred = malloc(sizeof(*cred)))) {
3184     						logerror("Unable to "
3185     							"allocate memory");
3186     						break;
3187     					}
>>>     CID 980936:    (USE_AFTER_FREE)
>>>     Using freed pointer "tmp_buf".
3188     					cred->data = tmp_buf;
3189     					tmp_buf = NULL;
3190     					SLIST_INSERT_HEAD(credhead,
3191     						cred, entries);
3192     				} while /* additional values? */
3193     					(copy_config_value_word(&tmp_buf, &p));
/usr.sbin/syslogd/syslogd.c: 3167 in read_config_file()
3161     				    config_keywords[i].keyword,
3162     				    *config_keywords[i].variable);
3163     #ifndef DISABLE_SIGN
3164     				if (!strcmp("sign_delim_sg2",
3165     				    config_keywords[i].keyword))
3166     					do {
>>>     CID 980936:    (USE_AFTER_FREE)
>>>     Calling "store_sign_delim_sg2(char *)" dereferences freed pointer "tmp_buf".
3167     						store_sign_delim_sg2(tmp_buf);
3168     					} while (copy_config_value_word(
3169     					    &tmp_buf, &p));
3170     
3171     #endif /* !DISABLE_SIGN */
3172     
/usr.sbin/syslogd/syslogd.c: 3167 in read_config_file()
3161     				    config_keywords[i].keyword,
3162     				    *config_keywords[i].variable);
3163     #ifndef DISABLE_SIGN
3164     				if (!strcmp("sign_delim_sg2",
3165     				    config_keywords[i].keyword))
3166     					do {
>>>     CID 980936:    (USE_AFTER_FREE)
>>>     Passing freed pointer "tmp_buf" as an argument to function "store_sign_delim_sg2(char *)".
3167     						store_sign_delim_sg2(tmp_buf);
3168     					} while (copy_config_value_word(
3169     					    &tmp_buf, &p));
3170     
3171     #endif /* !DISABLE_SIGN */
3172     


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://scan.coverity.com/projects/1449?tab=overview

To manage Coverity Scan email notifications for "coverity-updates%netbsd.org@localhost", click https://scan.coverity.com/subscriptions/edit?email=coverity-updates%40netbsd.org&token=487286ca1a9a4f4bd485d16f66b5e782 .