Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 8 Aug 2020 11:06:28 +0000 (UTC)
From:      Dimitry Andric <dim@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r364050 - head/tests/sys/net/routing
Message-ID:  <202008081106.078B6SGg041883@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: dim
Date: Sat Aug  8 11:06:27 2020
New Revision: 364050
URL: https://svnweb.freebsd.org/changeset/base/364050

Log:
  Use static inline for iface_{setup,delete}_addr in tests/sys/net/routing.
  
  This fixes possible link errors, similar to:
  
  ld: error: undefined symbol: iface_setup_addr
  >>> referenced by test_rtsock_l3.c:111 (tests/sys/net/routing/test_rtsock_l3.c:111)
  >>>               test_rtsock_l3.o:(presetup_ipv4)
  >>> referenced by test_rtsock_l3.c:79 (tests/sys/net/routing/test_rtsock_l3.c:79)
  >>>               test_rtsock_l3.o:(presetup_ipv6)
  >>> referenced by test_rtsock_l3.c:512 (tests/sys/net/routing/test_rtsock_l3.c:512)
  >>>               test_rtsock_l3.o:(atfu_rtm_change_v4_gw_success_body)
  >>> referenced 10 more times
  
  In C (not C++), 'naked' inline is almost always a mistake. Either use
  static inline (this is appropriate for most cases), or extern inline.
  
  MFC after:	3 days

Modified:
  head/tests/sys/net/routing/rtsock_common.h

Modified: head/tests/sys/net/routing/rtsock_common.h
==============================================================================
--- head/tests/sys/net/routing/rtsock_common.h	Sat Aug  8 10:05:27 2020	(r364049)
+++ head/tests/sys/net/routing/rtsock_common.h	Sat Aug  8 11:06:27 2020	(r364050)
@@ -204,7 +204,7 @@ iface_open(char *ifname)
  * Sets primary IPv4 addr.
  * Returns 0 on success.
  */
-inline int
+static inline int
 iface_setup_addr(char *ifname, char *addr, int plen)
 {
 	char cmd[512];
@@ -225,7 +225,7 @@ iface_setup_addr(char *ifname, char *addr, int plen)
  * Removes primary IPv4 prefix.
  * Returns 0 on success.
  */
-inline int
+static inline int
 iface_delete_addr(char *ifname, char *addr)
 {
 	char cmd[512];



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202008081106.078B6SGg041883>