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__)