From owner-svn-src-head@freebsd.org Wed Nov 29 12:16:07 2017 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 0DA8CE5254E; Wed, 29 Nov 2017 12:16:07 +0000 (UTC) (envelope-from o.hartmann@walstatt.org) Received: from mout.gmx.net (mout.gmx.net [212.227.17.20]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "mout.gmx.net", Issuer "TeleSec ServerPass DE-2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 7DDDB71C18; Wed, 29 Nov 2017 12:16:06 +0000 (UTC) (envelope-from o.hartmann@walstatt.org) Received: from hermann ([78.52.117.12]) by mail.gmx.com (mrgmx103 [212.227.17.168]) with ESMTPSA (Nemesis) id 0MQih7-1ehQnX02gv-00U0pv; Wed, 29 Nov 2017 13:16:02 +0100 Date: Wed, 29 Nov 2017 13:16:00 +0100 From: "Hartmann, O." To: Hans Petter Selasky Cc: "Hartmann, O." , svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r326362 - in head: share/man/man4 sys/net Message-ID: <20171129131556.2247c97c@hermann> In-Reply-To: References: <201711290940.vAT9eBWV096246@repo.freebsd.org> <20171129115125.24dd4aa0@hermann> Organization: walstatt.org MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:spfdgGKBeRO21cG8/Cna9FhHBPkYbMKOWPdMmOZBNbO5cAXBiYB cM36Eu3iP8d0/sxXOe+ktP8dXEmYCWuqH7NUAOUlpjOz11EMLfjQtI7xXqw+DCRBsI47JtV pXaIbK2+d7gU5XMiUvnr9Iub8Bw1qIOMecNV3HRy5bzzDzibx1ZFBhZEsgl5zQvePw6bVwa HiZ9l0i/Dd3ys+N64VftA== X-UI-Out-Filterresults: notjunk:1;V01:K0:WByoBd/3oAc=:TEJ79XdLz/wTD8GCcTlJ4N iR+QXCSJMsbquJdVQcO4z/6HaBLf2eVMD+68LDaQspR5Fc9aRz20VS0QbF0r8rPrytQU0gPWI FRRwheD+g45WeEG8xqk+AXx/J95rMQU7P9NrWvkr5iy1qcFQSuwiinmK2rqVivS+fTm6PDEJH eeDmNOngARxlHIKayeBq+NdMvzmwg0wf+g8t1/umvJTTrg0yynAMkM+oNjk9UkFWfKIQ60Ofu SJDwmh+S4KbEcu70FiAu4zpuFIaTE0YsLgGwUze/9kU7xkOx4r+fa+ijFmWuCKZ7+5AJfzqf6 EZdoXae74P75Tf5xbCVuDcFdREF+2Pngb1WROxRHaxsRBJQepPED0oHlT5+4TdS4AJsI/8Bzn +0rK439vunfpkO+vm+ny2/u5bVQ/7FbjzbHuWHUjdtmwD8L+YZ+7DlCmptAx0wx1j1tHE+g+B 7+yQQv7nvHM7g0LiDioFppqdWRPsKJdSJHgHE6IUdq4aFRAgWtlz3qIdzzPUHH3fYMRKUxgT3 Dw+F3bpYJI0IkxQN2zaL9xHnDsF7FfqCq/c4zpmFVgBoKWO8CVSoOflFjF7o0jWApL2hCCGjU Cu8UOqTA6YOg0vQydjkuzZ83RB4nGYwFIV8YZNLmaahUzUtkgX/sO/zGWUYgpP3zlggN/rPRr HFmIGfrbQk0FgrNcsln6QBv/J5umlu6z8MLwvJEpNnRia2BfpOq8TXBKgBj2WoKExrKO8xC5W CKNC271K5n3jZv6wMl1Xyph8WdQHSLaQVr+z3mjTjBKpQYPrk5haEj1ozw/nSLsNXGrOUIsPE bO8IjvdlDQF+OD1hMnx7tIzg3WVGHEWOUzSKb1/X753+szaeBY= X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 29 Nov 2017 12:16:07 -0000 On Wed, 29 Nov 2017 12:49:19 +0100 Hans Petter Selasky wrote: > On 11/29/17 11:51, Hartmann, O. wrote: > > On Wed, 29 Nov 2017 09:40:11 +0000 (UTC) > > Hans Petter Selasky wrote: > > > >> Author: hselasky > >> Date: Wed Nov 29 09:40:11 2017 > >> New Revision: 326362 > >> URL: https://svnweb.freebsd.org/changeset/base/326362 > >> > >> Log: > >> Disallow TUN and TAP character device IOCTLs to modify the > >> network device type to any value. This can cause page faults and > >> panics due to accessing uninitialized fields in the "struct ifnet" > >> which are specific to the network device type. > >> > >> MFC after: 1 week > >> Found by: jau@iki.fi > >> PR: 223767 > >> Sponsored by: Mellanox Technologies > >> > >> Modified: > >> head/share/man/man4/tap.4 > >> head/share/man/man4/tun.4 > >> head/sys/net/if_tap.c > >> head/sys/net/if_tun.c > >> > >> Modified: head/share/man/man4/tap.4 > >> ============================================================================== > >> --- head/share/man/man4/tap.4 Wed Nov 29 09:18:24 2017 > >> (r326361) +++ head/share/man/man4/tap.4 Wed Nov 29 09:40:11 > >> 2017 (r326362) @@ -1,7 +1,7 @@ > >> .\" $FreeBSD$ > >> .\" Based on PR#2411 > >> .\" > >> -.Dd April 10, 2015 > >> +.Dd November 29, 2017 > >> .Dt TAP 4 > >> .Os > >> .Sh NAME > >> @@ -171,7 +171,14 @@ calls are supported > >> .In net/if_tap.h ) : > >> .Bl -tag -width VMIO_SIOCSETMACADDR > >> .It Dv TAPSIFINFO > >> -Set network interface information (line speed, MTU and type). > >> +Set network interface information (line speed and MTU). > >> +The type must be the same as returned by > >> +.Dv TAPGIFINFO > >> +or set to > >> +.Dv IFT_ETHER > >> +else the > >> +.Xr ioctl 2 > >> +call will fail. > >> The argument should be a pointer to a > >> .Va struct tapinfo . > >> .It Dv TAPGIFINFO > >> > >> Modified: head/share/man/man4/tun.4 > >> ============================================================================== > >> --- head/share/man/man4/tun.4 Wed Nov 29 09:18:24 2017 > >> (r326361) +++ head/share/man/man4/tun.4 Wed Nov 29 09:40:11 > >> 2017 (r326362) @@ -2,7 +2,7 @@ > >> .\" $FreeBSD$ > >> .\" Based on PR#2411 > >> .\" > >> -.Dd November 30, 2014 > >> +.Dd November 29, 2017 > >> .Dt TUN 4 > >> .Os > >> .Sh NAME > >> @@ -208,8 +208,15 @@ this stores the internal debugging variable's > >> value in .It Dv TUNSIFINFO > >> The argument should be a pointer to an > >> .Vt struct tuninfo > >> -and allows setting the MTU, the type, and the baudrate of the > >> tunnel +and allows setting the MTU and the baudrate of the tunnel > >> device. > >> +The type must be the same as returned by > >> +.Dv TUNGIFINFO > >> +or set to > >> +.Dv IFT_PPP > >> +else the > >> +.Xr ioctl 2 > >> +call will fail. > >> The > >> .Vt struct tuninfo > >> is declared in > >> > >> Modified: head/sys/net/if_tap.c > >> ============================================================================== > >> --- head/sys/net/if_tap.c Wed Nov 29 09:18:24 2017 > >> (r326361) +++ head/sys/net/if_tap.c Wed Nov 29 09:40:11 > >> 2017 (r326362) @@ -737,9 +737,10 @@ tapioctl(struct cdev > >> *dev, u_long cmd, caddr_t data, i switch (cmd) { > >> case TAPSIFINFO: > >> tapp = (struct tapinfo *)data; > >> + if (ifp->if_type != tapp->type) > >> + return (EPROTOTYPE); > >> mtx_lock(&tp->tap_mtx); > >> ifp->if_mtu = tapp->mtu; > >> - ifp->if_type = tapp->type; > >> ifp->if_baudrate = tapp->baudrate; > >> mtx_unlock(&tp->tap_mtx); > >> break; > >> > >> Modified: head/sys/net/if_tun.c > >> ============================================================================== > >> --- head/sys/net/if_tun.c Wed Nov 29 09:18:24 2017 > >> (r326361) +++ head/sys/net/if_tun.c Wed Nov 29 09:40:11 > >> 2017 (r326362) @@ -676,9 +676,10 @@ tunioctl(struct cdev > >> *dev, u_long cmd, caddr_t data, i if (error) > >> return (error); > >> } > >> + if (TUN2IFP(tp)->if_type != tunp->type) > >> + return (EPROTOTYPE); > >> mtx_lock(&tp->tun_mtx); > >> TUN2IFP(tp)->if_mtu = tunp->mtu; > >> - TUN2IFP(tp)->if_type = tunp->type; > >> TUN2IFP(tp)->if_baudrate = tunp->baudrate; > >> mtx_unlock(&tp->tun_mtx); > >> break; > >> _______________________________________________ > >> svn-src-head@freebsd.org mailing list > >> https://lists.freebsd.org/mailman/listinfo/svn-src-head > >> To unsubscribe, send any mail to > >> "svn-src-head-unsubscribe@freebsd.org" > > > > after updating from r325893 to r326362, FreeBSD CURRENT crashes > > while booting the kernel. I'm sorry having no further informations, > > it happens on a laptop with reduced space. > > > > At the moment, it seems that a lot of boxes running most recent > > CURRENT tend to crash spontanously. > > > > Hi, > > And you built the kernel from scratch and made sure your source tree > does not contain any .o files nor /usr/obj/* . > > --HPS Last time I did the make cleandir was when the OFED driver problem occured, that was some days ago. Useually, I build world and kernel with WITH_META_MODE set. Sinverely oh