lacp: Fix misleading prototype for lacp_configure().
authorBen Pfaff <blp@nicira.com>
Mon, 21 Mar 2011 20:15:31 +0000 (13:15 -0700)
committerBen Pfaff <blp@nicira.com>
Fri, 1 Apr 2011 22:52:20 +0000 (15:52 -0700)
Only the first 6 bytes (ETH_ADDR_LEN) of the 'sys_id' argument are used,
but the prototype declared it as an array of 8 bytes.  This has no effect
on the generated code--the declared size of an array parameter is
irrelevant--but it is misleading.

Also, add 'const' since the array is not modified.

lib/lacp.c
lib/lacp.h

index e177a38afd87cdbdd83b486aa4db2456d902bbfb..005165fdd7d7f630e58ad9e4a5160c81a30fc6ed 100644 (file)
@@ -128,7 +128,7 @@ lacp_destroy(struct lacp *lacp)
  * 'active' parameters. */
 void
 lacp_configure(struct lacp *lacp, const char *name,
-               uint8_t sys_id[ETH_ADDR_LEN], uint16_t sys_priority,
+               const uint8_t sys_id[ETH_ADDR_LEN], uint16_t sys_priority,
                bool active, bool fast)
 {
     if (!lacp->name || strcmp(name, lacp->name)) {
index 60abaddeb8aa8653e41b2d01f1c4823dacb16202..31ac9705eea1b87fa8a3c06ccd46ceda6e9f5a97 100644 (file)
@@ -19,6 +19,7 @@
 
 #include <stdbool.h>
 #include <stdint.h>
+#include "packets.h"
 
 struct ds;
 struct lacp;
@@ -31,7 +32,8 @@ typedef void lacp_send_pdu(void *slave, const struct lacp_pdu *);
 void lacp_init(void);
 struct lacp *lacp_create(void);
 void lacp_destroy(struct lacp *);
-void lacp_configure(struct lacp *, const char *name, uint8_t sys_id[8],
+void lacp_configure(struct lacp *, const char *name,
+                    const uint8_t sys_id[ETH_ADDR_LEN],
                     uint16_t sys_priority, bool active, bool fast);
 void lacp_process_pdu(struct lacp *, const void *slave,
                       const struct lacp_pdu *);