Hi,

in the function 'mpm_service_install' in server/mpm/winnt/service.c, cppcheck complains on line 596 and line 608 about 'Same expression on both sides of '|'.'

596: ap_log_error(APLOG_MARK, APLOG_ERR|APLOG_ERR,<--- Same expression on both sides of '|'.
                         apr_get_os_error(), NULL,
                         "OpenService failed");

608: ap_log_error(APLOG_MARK, APLOG_ERR|APLOG_ERR,<--- Same expression on both sides of '|'.
                         apr_get_os_error(), NULL,
                         "ChangeServiceConfig failed");


According to surrounding code, I assume that the second APLOG_ERR should be APLOG_STARTUP.

Any idea ?


Best regards,
CJ


Reply via email to