Module Name:    src
Committed By:   pgoyette
Date:           Fri Sep 27 00:32:04 UTC 2019

Modified Files:
        src/sys/compat/common: kern_uipc_socket_50.c
        src/sys/kern: uipc_socket.c
        src/sys/sys: compat_stub.h param.h

Log Message:
Actually return the updated pointer-to-mbuf-pointer to the caller
rather than discarding-after-assignment.  Introduced from the
[pgoyette-compat] branch work.

Welcome to 9.99.14 !!!  (Module hook routine prototype changed.)

Found by the lgtm bot, reported via private Email from maxv@


To generate a diff of this commit:
cvs rdiff -u -r1.2 -r1.3 src/sys/compat/common/kern_uipc_socket_50.c
cvs rdiff -u -r1.283 -r1.284 src/sys/kern/uipc_socket.c
cvs rdiff -u -r1.18 -r1.19 src/sys/sys/compat_stub.h
cvs rdiff -u -r1.613 -r1.614 src/sys/sys/param.h

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/compat/common/kern_uipc_socket_50.c
diff -u src/sys/compat/common/kern_uipc_socket_50.c:1.2 src/sys/compat/common/kern_uipc_socket_50.c:1.3
--- src/sys/compat/common/kern_uipc_socket_50.c:1.2	Mon Apr 15 10:53:17 2019
+++ src/sys/compat/common/kern_uipc_socket_50.c	Fri Sep 27 00:32:03 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: kern_uipc_socket_50.c,v 1.2 2019/04/15 10:53:17 pgoyette Exp $	*/
+/*	$NetBSD: kern_uipc_socket_50.c,v 1.3 2019/09/27 00:32:03 pgoyette Exp $	*/
 
 /*
  * Copyright (c) 2002, 2007, 2008, 2009 The NetBSD Foundation, Inc.
@@ -101,7 +101,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: kern_uipc_socket_50.c,v 1.2 2019/04/15 10:53:17 pgoyette Exp $");
+__KERNEL_RCSID(0, "$NetBSD: kern_uipc_socket_50.c,v 1.3 2019/09/27 00:32:03 pgoyette Exp $");
 
 #if defined(_KERNEL_OPT)
 #include "opt_compat_netbsd.h"
@@ -210,7 +210,7 @@ uipc_socket_50_setopt1(int opt, struct s
 }
 
 static int
-uipc_socket_50_sbts(int opt, struct mbuf **mp)
+uipc_socket_50_sbts(int opt, struct mbuf ***mp)
 {
 	struct timeval50 tv50;
 	struct timeval tv;
@@ -220,10 +220,10 @@ uipc_socket_50_sbts(int opt, struct mbuf
 	if (opt & SO_OTIMESTAMP) {
 
 		timeval_to_timeval50(&tv, &tv50);
-		*mp = sbcreatecontrol(&tv50, sizeof(tv50), SCM_OTIMESTAMP,
+		**mp = sbcreatecontrol(&tv50, sizeof(tv50), SCM_OTIMESTAMP,
 		    SOL_SOCKET);
-		if (*mp)
-			mp = &(*mp)->m_next;
+		if (**mp)
+			*mp = &(**mp)->m_next;
 		return 0;
 	} else
 		return EPASSTHROUGH;

Index: src/sys/kern/uipc_socket.c
diff -u src/sys/kern/uipc_socket.c:1.283 src/sys/kern/uipc_socket.c:1.284
--- src/sys/kern/uipc_socket.c:1.283	Sat Sep 14 15:06:33 2019
+++ src/sys/kern/uipc_socket.c	Fri Sep 27 00:32:03 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: uipc_socket.c,v 1.283 2019/09/14 15:06:33 mlelstv Exp $	*/
+/*	$NetBSD: uipc_socket.c,v 1.284 2019/09/27 00:32:03 pgoyette Exp $	*/
 
 /*
  * Copyright (c) 2002, 2007, 2008, 2009 The NetBSD Foundation, Inc.
@@ -71,7 +71,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: uipc_socket.c,v 1.283 2019/09/14 15:06:33 mlelstv Exp $");
+__KERNEL_RCSID(0, "$NetBSD: uipc_socket.c,v 1.284 2019/09/27 00:32:03 pgoyette Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_compat_netbsd.h"
@@ -2419,7 +2419,7 @@ sbsavetimestamp(int opt, struct mbuf **m
 
 	microtime(&tv);
 
-	MODULE_HOOK_CALL(uipc_socket_50_sbts_hook, (opt, mp), enosys(), error);
+	MODULE_HOOK_CALL(uipc_socket_50_sbts_hook, (opt, &mp), enosys(), error);
 	if (error == 0)
 		return mp;
 

Index: src/sys/sys/compat_stub.h
diff -u src/sys/sys/compat_stub.h:1.18 src/sys/sys/compat_stub.h:1.19
--- src/sys/sys/compat_stub.h:1.18	Sat Jul 20 20:59:05 2019
+++ src/sys/sys/compat_stub.h	Fri Sep 27 00:32:03 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: compat_stub.h,v 1.18 2019/07/20 20:59:05 mrg Exp $	*/
+/*	$NetBSD: compat_stub.h,v 1.19 2019/09/27 00:32:03 pgoyette Exp $	*/
 
 /*-
  * Copyright (c) 2018 The NetBSD Foundation, Inc.
@@ -231,7 +231,7 @@ MODULE_HOOK(uipc_socket_50_setopt1_hook,
     (int, struct socket *, const struct sockopt *));
 MODULE_HOOK(uipc_socket_50_getopt1_hook, int,
     (int, struct socket *, struct sockopt *));
-MODULE_HOOK(uipc_socket_50_sbts_hook, int, (int, struct mbuf **));
+MODULE_HOOK(uipc_socket_50_sbts_hook, int, (int, struct mbuf ***));
 
 /*
  * uipc_syscalls_50 compatibility

Index: src/sys/sys/param.h
diff -u src/sys/sys/param.h:1.613 src/sys/sys/param.h:1.614
--- src/sys/sys/param.h:1.613	Sun Sep 22 22:59:40 2019
+++ src/sys/sys/param.h	Fri Sep 27 00:32:03 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: param.h,v 1.613 2019/09/22 22:59:40 christos Exp $	*/
+/*	$NetBSD: param.h,v 1.614 2019/09/27 00:32:03 pgoyette Exp $	*/
 
 /*-
  * Copyright (c) 1982, 1986, 1989, 1993
@@ -67,7 +67,7 @@
  *	2.99.9		(299000900)
  */
 
-#define	__NetBSD_Version__	999001300	/* NetBSD 9.99.13 */
+#define	__NetBSD_Version__	999001400	/* NetBSD 9.99.14 */
 
 #define __NetBSD_Prereq__(M,m,p) (((((M) * 100000000) + \
     (m) * 1000000) + (p) * 100) <= __NetBSD_Version__)

Reply via email to