safestringlib: silence unused parameters warning in error handlers

The default ignore and abort handlers take ptr parameter which is unused, in addition in the ignore_handler we use debug print macro which might be undefined and render also rest of the parameters unused. Introduce UNUSED() macro that silence the warning. Signed-off-by: Tomas Winkler <[email protected]>
intel · Feb 12, 2024 · ed893c2 · ed893c2
1 parent 4ee034f
commit ed893c2
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/safeclib/abort_handler_s.c b/safeclib/abort_handler_s.c
@@ -44,6 +44,8 @@
 
 void abort_handler_s(const char *msg, void *ptr, errno_t error)
 {
+    UNUSED(ptr);
+
     slprintf("ABORT CONSTRAINT HANDLER: (%u) %s\n", error,
          (msg) ? msg : "Null message");
     slabort();

diff --git a/safeclib/ignore_handler_s.c b/safeclib/ignore_handler_s.c
@@ -40,9 +40,11 @@
 
 void ignore_handler_s(const char *msg, void *ptr, errno_t error)
 {
+    UNUSED(ptr);
+    UNUSED(msg);
+    UNUSED(error);
 
     sldebug_printf("IGNORE CONSTRAINT HANDLER: (%u) %s\n", error,
                (msg) ? msg : "Null message");
-    return;
 }
 EXPORT_SYMBOL(ignore_handler_s)
diff --git a/safeclib/safeclib_private.h b/safeclib/safeclib_private.h
@@ -60,6 +60,8 @@
 
 #endif /* __KERNEL__ */
 
+#define UNUSED(x) (void)(x)
+
 #ifndef sldebug_printf
 #define sldebug_printf(...)
 #endif