RIC-641 Fixing client/server model definitions and adding client and server API

[ric-plt/xapp-frame-cpp.git] / src / messaging / default_cb.cpp

diff --git a/src/messaging/default_cb.cpp b/src/messaging/default_cb.cpp
index 1083293..681f5f6 100644 (file)
--- a/src/messaging/default_cb.cpp
+++ b/src/messaging/default_cb.cpp
@@ -38,6 +38,9 @@
 
 #include "messenger.hpp"
 
+namespace xapp {
+
+
 /*
        This is the default health check function that we provide (user
        may override it).  It will respond to health check messages by
@@ -45,10 +48,21 @@
 
        The mr paramter is obviously ignored, but to add this as a callback
        the function sig must match.
+
+       This is a callback function; sonar will complain that we don't use payload or
+       data -- we don't, but this is a standard function proto so we cannot just
+       drop them.
 */
 void Health_ck_cb( Message& mbuf, int mtype, int sid, int len, Msg_component payload, void* data ) {
        unsigned char response[128];
 
+       if( len < 0  || mtype < 0 ) {
+               return;
+       }
+
        snprintf( (char* ) response, sizeof( response ), "OK\n" );
        mbuf.Send_response( RIC_HEALTH_CHECK_RESP, sid, strlen( (char *) response )+1, response );
 }
+
+
+} // namespace