*: Remove int return for zapi functions

The code never uses the return code for Zapi functions. Let's remove it. Signed-off-by: Donald Sharp <[email protected]>
FRRouting · Oct 10, 2024 · 2020be2 · 2020be2
1 parent cc7d89a
commit 2020be2
Show file tree

Hide file tree

Showing 32 changed files with 388 additions and 619 deletions.
diff --git a/babeld/babel_interface.c b/babeld/babel_interface.c
@@ -88,8 +88,7 @@ babel_ifp_destroy(struct interface *ifp)
     return 0;
 }
 
-int
-babel_interface_address_add (ZAPI_CALLBACK_ARGS)
+void babel_interface_address_add(ZAPI_CALLBACK_ARGS)
 {
     babel_interface_nfo *babel_ifp;
     struct connected *ifc;
@@ -101,7 +100,7 @@ babel_interface_address_add (ZAPI_CALLBACK_ARGS)
                                         zclient->ibuf, vrf_id);
 
     if (ifc == NULL)
-        return 0;
+	    return;
 
     prefix = ifc->address;
 
@@ -121,12 +120,9 @@ babel_interface_address_add (ZAPI_CALLBACK_ARGS)
 
     send_request(ifc->ifp, NULL, 0);
     send_update(ifc->ifp, 0, NULL, 0);
-
-    return 0;
 }
 
-int
-babel_interface_address_delete (ZAPI_CALLBACK_ARGS)
+void babel_interface_address_delete(ZAPI_CALLBACK_ARGS)
 {
     babel_interface_nfo *babel_ifp;
     struct connected *ifc;
@@ -138,7 +134,7 @@ babel_interface_address_delete (ZAPI_CALLBACK_ARGS)
                                         zclient->ibuf, vrf_id);
 
     if (ifc == NULL)
-        return 0;
+	    return;
 
     prefix = ifc->address;
 
@@ -158,7 +154,6 @@ babel_interface_address_delete (ZAPI_CALLBACK_ARGS)
     send_update(ifc->ifp, 0, NULL, 0);
 
     connected_free(&ifc);
-    return 0;
 }
 
 /* Lookup function. */

diff --git a/babeld/babel_interface.h b/babeld/babel_interface.h
@@ -97,12 +97,12 @@ void babel_if_init(void);
 void babel_if_terminate(void);
 
 /* Callback functions for zebra client */
-int babel_interface_up (int, struct zclient *, zebra_size_t, vrf_id_t);
-int babel_interface_down (int, struct zclient *, zebra_size_t, vrf_id_t);
-int babel_interface_add (int, struct zclient *, zebra_size_t, vrf_id_t);
-int babel_interface_delete (int, struct zclient *, zebra_size_t, vrf_id_t);
-int babel_interface_address_add (int, struct zclient *, zebra_size_t, vrf_id_t);
-int babel_interface_address_delete (int, struct zclient *, zebra_size_t, vrf_id_t);
+void babel_interface_up(ZAPI_CALLBACK_ARGS);
+void babel_interface_down(ZAPI_CALLBACK_ARGS);
+void babel_interface_add(ZAPI_CALLBACK_ARGS);
+void babel_interface_delete(ZAPI_CALLBACK_ARGS);
+void babel_interface_address_add(ZAPI_CALLBACK_ARGS);
+void babel_interface_address_delete(ZAPI_CALLBACK_ARGS);
 
 int babel_ifp_create(struct interface *ifp);
 int babel_ifp_up(struct interface *ifp);

diff --git a/babeld/babel_zebra.c b/babeld/babel_zebra.c
@@ -38,25 +38,22 @@ static const struct {
 };
 
 /* Zebra route add and delete treatment. */
-static int
-babel_zebra_read_route (ZAPI_CALLBACK_ARGS)
+static void babel_zebra_read_route(ZAPI_CALLBACK_ARGS)
 {
     struct zapi_route api;
 
     if (zapi_route_decode(zclient->ibuf, &api) < 0)
-        return -1;
+	    return;
 
     /* we completely ignore srcdest routes for now. */
     if (CHECK_FLAG(api.message, ZAPI_MESSAGE_SRCPFX))
-        return 0;
+	    return;
 
     if (cmd == ZEBRA_REDISTRIBUTE_ROUTE_ADD) {
         babel_route_add(&api);
     } else {
         babel_route_delete(&api);
     }
-
-    return 0;
 }
 
 /* [Babel Command] */

diff --git a/bfdd/ptm_adapter.c b/bfdd/ptm_adapter.c
@@ -588,7 +588,7 @@ static void bfdd_client_deregister(struct stream *msg)
 	zlog_err("ptm-del-client: failed to deregister client");
 }
 
-static int bfdd_replay(ZAPI_CALLBACK_ARGS)
+static void bfdd_replay(ZAPI_CALLBACK_ARGS)
 {
 	struct stream *msg = zclient->ibuf;
 	uint32_t rcmd;
@@ -613,14 +613,11 @@ static int bfdd_replay(ZAPI_CALLBACK_ARGS)
 	default:
 		if (bglobal.debug_zebra)
 			zlog_debug("ptm-replay: invalid message type %u", rcmd);
-		return -1;
+		return;
 	}
 
-	return 0;
-
 stream_failure:
 	zlog_err("ptm-replay: failed to find command");
-	return -1;
 }
 
 static void bfdd_zebra_connected(struct zclient *zc)
@@ -778,13 +775,13 @@ static void bfdd_sessions_enable_address(struct connected *ifc)
 	}
 }
 
-static int bfdd_interface_address_update(ZAPI_CALLBACK_ARGS)
+static void bfdd_interface_address_update(ZAPI_CALLBACK_ARGS)
 {
 	struct connected *ifc;
 
 	ifc = zebra_interface_address_read(cmd, zclient->ibuf, vrf_id);
 	if (ifc == NULL)
-		return 0;
+		return;
 
 	if (bglobal.debug_zebra)
 		zlog_debug("zclient: %s local address %pFX (VRF %u)",
@@ -796,8 +793,6 @@ static int bfdd_interface_address_update(ZAPI_CALLBACK_ARGS)
 		bfdd_sessions_enable_address(ifc);
 	else
 		connected_free(&ifc);
-
-	return 0;
 }
 
 static int bfd_ifp_create(struct interface *ifp)