Projects
Mega:24.09
audit
_service:tar_scm:backport-Fix-new-warnings-for-...
Sign Up
Log In
Username
Password
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File _service:tar_scm:backport-Fix-new-warnings-for-unused-results.patch of Package audit
From a4e8b7e18f249fe5decdd2fe748a5068ffeaee57 Mon Sep 17 00:00:00 2001 From: Steve Grubb <ausearch.1@gmail.com> Date: Mon, 20 Nov 2023 16:37:46 -0500 Subject: [PATCH] Fix new warnings for unused results Reference:https://github.com/linux-audit/audit-userspace/commit/a4e8b7e18f249fe5decdd2fe748a5068ffeaee57 Conflict:NA --- audisp/plugins/ids/ids.c | 5 +++-- audisp/plugins/ids/ids.h | 2 +- audisp/plugins/statsd/audisp-statsd.c | 4 ++-- lib/libaudit.c | 3 ++- lib/netlink.c | 3 ++- src/auditd.c | 3 ++- 6 files changed, 12 insertions(+), 8 deletions(-) diff --git a/audisp/plugins/ids/ids.c b/audisp/plugins/ids/ids.c index d28237e5..1446ca71 100644 --- a/audisp/plugins/ids/ids.c +++ b/audisp/plugins/ids/ids.c @@ -107,9 +107,10 @@ static void destroy_audit(void) } -void log_audit_event(int type, const char *text, int res) +int log_audit_event(int type, const char *text, int res) { - audit_log_user_message(audit_fd, type, text, NULL, NULL, NULL, res); + return audit_log_user_message(audit_fd, type, text, + NULL, NULL, NULL, res); } diff --git a/audisp/plugins/ids/ids.h b/audisp/plugins/ids/ids.h index f3710066..cb98cdba 100644 --- a/audisp/plugins/ids/ids.h +++ b/audisp/plugins/ids/ids.h @@ -15,6 +15,6 @@ extern int debug; extern void my_printf(const char *fmt, ...) __attribute__ (( format(printf, 1, 2) )); -extern void log_audit_event(int type, const char *text, int res); +extern int log_audit_event(int type, const char *text, int res); #endif diff --git a/audisp/plugins/statsd/audisp-statsd.c b/audisp/plugins/statsd/audisp-statsd.c index db2c6111..912f9171 100644 --- a/audisp/plugins/statsd/audisp-statsd.c +++ b/audisp/plugins/statsd/audisp-statsd.c @@ -218,9 +218,9 @@ static void get_kernel_status(void) struct audit_reply rep; audit_request_status(audit_fd); - audit_get_reply(audit_fd, &rep, GET_REPLY_BLOCKING, 0); + int rc = audit_get_reply(audit_fd, &rep, GET_REPLY_BLOCKING, 0); - if (rep.type == AUDIT_GET) { + if (rc > 0 && rep.type == AUDIT_GET) { // add info to global audit event struct r.lost = rep.status->lost; r.backlog = rep.status->backlog; diff --git a/lib/libaudit.c b/lib/libaudit.c index e5f2a7c5..3decff12 100644 --- a/lib/libaudit.c +++ b/lib/libaudit.c @@ -473,7 +473,8 @@ int audit_set_pid(int fd, uint32_t pid, rep_wait_t wmode) rc = poll(pfd, 1, 100); /* .1 second */ } while (rc < 0 && errno == EINTR); - (void)audit_get_reply(fd, &rep, GET_REPLY_NONBLOCKING, 0); + if (audit_get_reply(fd, &rep, GET_REPLY_NONBLOCKING, 0)) + ; // intentionally empty return 1; } diff --git a/lib/netlink.c b/lib/netlink.c index eeeefc26..3381651a 100644 --- a/lib/netlink.c +++ b/lib/netlink.c @@ -280,7 +280,8 @@ retry: else if (rc > 0 && rep.type == NLMSG_ERROR) { int error = rep.error->error; /* Eat the message */ - (void)audit_get_reply(fd, &rep, GET_REPLY_NONBLOCKING, 0); + if (audit_get_reply(fd, &rep, GET_REPLY_NONBLOCKING, 0)) + ; // intentionally empty /* NLMSG_ERROR can indicate success, only report nonzero */ if (error) { diff --git a/src/auditd.c b/src/auditd.c index 2dedf35b..54b407f3 100644 --- a/src/auditd.c +++ b/src/auditd.c @@ -1044,7 +1044,8 @@ static void clean_exit(void) audit_msg(LOG_INFO, "The audit daemon is exiting."); if (fd >= 0) { if (!opt_aggregate_only) - audit_set_pid(fd, 0, WAIT_NO); + if (audit_set_pid(fd, 0, WAIT_NO)) + ; // intentionally empty audit_close(fd); } if (pidfile) -- 2.33.0
Locations
Projects
Search
Status Monitor
Help
Open Build Service
OBS Manuals
API Documentation
OBS Portal
Reporting a Bug
Contact
Mailing List
Forums
Chat (IRC)
Twitter
Open Build Service (OBS)
is an
openSUSE project
.
浙ICP备2022010568号-2