]> git.puffer.fish Git - mirror/frr.git/commitdiff
bgpd: bgp_packet_set_size int to void 10147/head
authorDonatas Abraitis <donatas.abraitis@gmail.com>
Mon, 29 Nov 2021 19:23:53 +0000 (21:23 +0200)
committerDonatas Abraitis <donatas.abraitis@gmail.com>
Mon, 29 Nov 2021 19:23:53 +0000 (21:23 +0200)
stream size is never checked anywhere in the code, just convert to void.

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
bgpd/bgp_packet.c
bgpd/bgp_packet.h

index 319213ef76db3468e43ff0a39a447173edb86fa0..cc0ee9e8f26f2a7cc11bb5f130009a75a0e9cc29 100644 (file)
@@ -106,17 +106,14 @@ int bgp_packet_set_marker(struct stream *s, uint8_t type)
  * Size field is set to the size of the stream passed.
  *
  * @param s the stream containing the packet
- * @return the size of the stream
  */
-int bgp_packet_set_size(struct stream *s)
+void bgp_packet_set_size(struct stream *s)
 {
        int cp;
 
        /* Preserve current pointer. */
        cp = stream_get_endp(s);
        stream_putw_at(s, BGP_MARKER_SIZE, cp);
-
-       return cp;
 }
 
 /*
@@ -561,7 +558,7 @@ void bgp_keepalive_send(struct peer *peer)
        bgp_packet_set_marker(s, BGP_MSG_KEEPALIVE);
 
        /* Set packet size. */
-       (void)bgp_packet_set_size(s);
+       bgp_packet_set_size(s);
 
        /* Dump packet if debug option is set. */
        /* bgp_packet_dump (s); */
@@ -626,7 +623,7 @@ void bgp_open_send(struct peer *peer)
        }
 
        /* Set BGP packet length. */
-       (void)bgp_packet_set_size(s);
+       bgp_packet_set_size(s);
 
        if (bgp_debug_neighbor_events(peer))
                zlog_debug(
@@ -953,7 +950,7 @@ void bgp_route_refresh_send(struct peer *peer, afi_t afi, safi_t safi,
                }
 
        /* Set packet size. */
-       (void)bgp_packet_set_size(s);
+       bgp_packet_set_size(s);
 
        if (bgp_debug_neighbor_events(peer)) {
                if (!orf_refresh)
@@ -1011,7 +1008,7 @@ void bgp_capability_send(struct peer *peer, afi_t afi, safi_t safi,
        }
 
        /* Set packet size. */
-       (void)bgp_packet_set_size(s);
+       bgp_packet_set_size(s);
 
        /* Add packet to the peer. */
        bgp_packet_add(peer, s);
index d69c86230439fc038675614981319ac9525e9791..280d3ec1747624c8124fa1899296703f8c74f427 100644 (file)
@@ -77,7 +77,7 @@ extern void bgp_update_implicit_eors(struct peer *);
 extern void bgp_check_update_delay(struct bgp *);
 
 extern int bgp_packet_set_marker(struct stream *s, uint8_t type);
-extern int bgp_packet_set_size(struct stream *s);
+extern void bgp_packet_set_size(struct stream *s);
 
 extern int bgp_generate_updgrp_packets(struct thread *);
 extern int bgp_process_packet(struct thread *);