{"push_id":1141793911,"size":35,"distinct_size":35,"ref":"refs/heads/freebsd10","head":"89c240f5e5c714d17f1a883e0eb4707e3b1cd07c","before":"4d7d5867d643ddb8cd01c9279f25b91d7a3e01a1","commits":[{"sha":"8d1337c667c209cb68da838288130a25cf3441d6","author":{"name":"glebius","email":"80e534fe7e1b330136d3a302de2261cd031ee58f@FreeBSD.org"},"message":"Merge r301053:\n\n  Fix kernel stack disclosures in the Linux and 4.3BSD compat layers.\n\nSecurity:\tSA-16:20\nSecurity:\tSA-16:21","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/8d1337c667c209cb68da838288130a25cf3441d6"},{"sha":"0dfbd8e3cc8b9a97e27d9fb920bada3a7d2d5340","author":{"name":"ian","email":"668fd45181c31193793d12a7f2ecb1f2376600d1@FreeBSD.org"},"message":"MFC r297147, r297148, r297149, r297150, r297151:\n\n  Make both the loader and kernel use the interface-mtu option if the\n  dhcp server provides it.  Made up of these (semi-)related changes...\n\n  [kernel...] If the dhcp server provides an interface-mtu option, parse\n  the value and set that mtu on the interface.\n\n  [libstand...]\n\n  Garbage collect the bswap routines from libstand, use sys/endian.h.\n\n  If the dhcp server delivers an interface-mtu option, parse it and store\n  the value in a new global intf_mtu for use by the application.\n\n  [loader...]\n\n  If the dhcp server provided an interface-mtu option, transcribe the value\n  to the boot.netif.mtu env var, which will be picked up by pre-existing code\n  in nfs_mountroot() and used to configure the interface accordingly.\n\nPR:\t\t187094","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/0dfbd8e3cc8b9a97e27d9fb920bada3a7d2d5340"},{"sha":"801afed4d05a94621bc993ab5c2982b1a3bc27ae","author":{"name":"ian","email":"668fd45181c31193793d12a7f2ecb1f2376600d1@FreeBSD.org"},"message":"MFC r297323,r297324, r297325, r297326:\n\n  Set only one default route for nfsroot mount, the one associated with the\n  interface that will be used to mount the rootfs (and never a self-ip proxy\n  arp route).  Made up of the following related changes...\n\n  Set ifctx->gotrootpath=1 only when the root path came from the dhcp/bootp\n  server (and not when it came from a fallback method such as the ROOTDEVNAME\n  option).  This makes the code in bootpc_init() choose the first interface\n  that provided a rootpath name.  Previously it was choosing the first\n  interface that got an IP address, which could be on a different and\n  potentially unreachable subnet than the server providing the rootfs.\n\n  If the rootpath name actually does come from a fallback source, then the\n  code continues to use the first interface in the list that got configured.\n  Note that this wasn't directly reported in the PR cited below, but was\n  discovered while working on that PR.\n\n  Switch bootpc_adjust_interface() from returning int to void.  Its one caller\n  doesn't check for errors, and all the errors that can happen result in it\n  calling panic anyway, except for one that's really more of a warning (and\n  is going to disappear on an upcoming commit anyway).\n\n  Stop setting the default route to the IP of the interface itself when the\n  bootp/dhcp server doesn't provide a router option.  Doing so prevents\n  setting defaultrouter=<ip> in rc.conf (it fails because there's already\n  a bogus default route installed by bootpc_init).\n\n  When an admin wants to use this style of proxy arp on an interface, the\n  proper mechanism is to set the \"use-lease-addr-for-default-route\" flag\n  in the dhcp server config.  That causes the lease address to be delivered\n  in the routers option, and the normal handling of the routers option will\n  then install the self-ip as the default route.\n\n  Do not try to install a default route for each interface found, because\n  only the first one will actually work and all the others just result in\n  errors (which would get printed but otherwise ignored).\n\n  Instead, wait until we make a choice of which interface will be used to\n  mount the rootfs, and install the default route associated with it (if any).\n  After doing the md_mount() call to obtain the needed info, remove the\n  default route again, and transcribe the route info into the nfs_diskless\n  structure.  If the system eventually chooses to mount the nfs rootfs, the\n  default route will be installed again when the nfs_diskless code\n  re-initializes the interface.\n\nPR:\t\t187094","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/801afed4d05a94621bc993ab5c2982b1a3bc27ae"},{"sha":"7fcf94d717b75ca78552fdaa21dce3b316ab6bef","author":{"name":"kib","email":"ce9d581103e40ce1b4a3636d5281c849530292f7@FreeBSD.org"},"message":"MFC r300596:\nIn vm_page_alloc_contig(), on vm_page_insert() failure, mark each\nfreed page as VPO_UNMANAGED.  Otherwise vm_pge_free_toq() insists on\nowning the page lock.","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/7fcf94d717b75ca78552fdaa21dce3b316ab6bef"},{"sha":"0c5ab4bf42989bee63113415c0f2d7be8a2e30ed","author":{"name":"kib","email":"ce9d581103e40ce1b4a3636d5281c849530292f7@FreeBSD.org"},"message":"MFC r300622:\nIn vm_page_cache(), only drop the vnode after radix insert failure\nfor empty page cache when the object type is OBJT_VNODE.","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/0c5ab4bf42989bee63113415c0f2d7be8a2e30ed"},{"sha":"107f0031171e58052a12ca5d9d91e959234f7d13","author":{"name":"kib","email":"ce9d581103e40ce1b4a3636d5281c849530292f7@FreeBSD.org"},"message":"MFC r300142:\nEnsure that ftruncate(2) is performed synchronously when file is\nopened in O_SYNC mode, at least for UFS.","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/107f0031171e58052a12ca5d9d91e959234f7d13"},{"sha":"b179a063218f757e3c9d2a9c7c7941a36ea0f1af","author":{"name":"cy","email":"8e7af2df40968db17bcf20142c9fe3377726ad01@FreeBSD.org"},"message":"MFC r300180:\n\nUpdate leap-seconds to leap-seconds.3661459200.\nNO leap second will be introduced at the end of June 2016.\n\nObtained from:\tftp://tycho.usno.navy.mil/pub/ntp/.\nSee also:\thttp://www.iers.org/SharedDocs/News/EN/BulletinC.html","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/b179a063218f757e3c9d2a9c7c7941a36ea0f1af"},{"sha":"f28839d2d76920726759ae0f3b79fb01d250bf9b","author":{"name":"n_hibma","email":"f47b6971a884054708ae1f7214586aefec062bcc@FreeBSD.org"},"message":"MFC 300670 (slightly adapted for 10-STABLE):\n\nChange net.link.log_promisc_mode_change to a read-only tunable.\n\nPR:\t\t166255\nSubmitted by:\teugen.grosbein.net\nObtained from:\thselasky","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/f28839d2d76920726759ae0f3b79fb01d250bf9b"},{"sha":"841ce9904cc8019ed9d546625507a029293380a6","author":{"name":"truckman","email":"7db9b6fb22bf58dcc1fd7606fbae5570b479b7be@FreeBSD.org"},"message":"MFC r300442\n\nHopefully fix Coverity CID 1008328 (Out-of-bounds write) in /bin/sh.\n\nReplace the magic constant 127 in the loop interation count with\n\"PROMPTLEN - 1\".\n\ngethostname() is not guaranteed to NUL terminate the destination\nstring if it is too short. Decrease the length passed to gethostname()\nby one, and add a NUL at the end of the buffer to make sure the\nfollowing loop to find the end of the name properly terminates.\n\nThe default: case is the likely cause of Coverity CID 1008328.  If\ni is 126 at the top of the loop interation where the default case\nis triggered, i will be incremented to 127 by the default case,\nthen incremented to 128 at the top of the loop before being compared\nto 127 (PROMPTLENT - 1) and terminating the loop. Then the NUL\ntermination code after the loop will write to ps[128].  Fix by\nchecking for overflow before incrementing the index and storing the\nsecond character in the buffer.\n\nThese fixes are not guaranteed to satisfy Coverity. The code that\nincrements i in the 'h'/'H' and 'w'/'W' cases may be beyond its\ncapability to analyze, but the code appears to be safe.\n\nReported by:\tCoverity\nCID:\t\t1008328\nReviewed by:\tjilles, cem\nDifferential Revision:\thttps://reviews.freebsd.org/D6482","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/841ce9904cc8019ed9d546625507a029293380a6"},{"sha":"953b9044a7235743f9ebc263e5ec8472f755871e","author":{"name":"truckman","email":"7db9b6fb22bf58dcc1fd7606fbae5570b479b7be@FreeBSD.org"},"message":"MFC r300632\n\nFix acpidump CID 1011278 (Buffer not null terminated) and other issues\n\nCoverity reports that a buffer used for temporary file generation\nmight not be NUL terminated by strncpy().  This is probably not\ntrue because the input gets passed through realpath(), but if the\npath name is sufficiently long the name could be truncated and cause\nother problems.  The code for generating the temp file names is\nalso overly complex.  Instead of a bunch of calls to strncpy() and\nand strncat(), simplify the code by using snprintf() and add checks\nfor unexpected truncation.\n\nThe output file created by iasl -d is predictable.  Fix this by\nusing  mkdtemp() to create a directory to hold the iasl input and\noutput files.\n\nCheck the return values of more syscalls.\n\nReported by:\tCoverity\nCID:\t\t1011278\nReviewed by:\tjkim\nDifferential Revision:\thttps://reviews.freebsd.org/D6360","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/953b9044a7235743f9ebc263e5ec8472f755871e"},{"sha":"a88b1b7c540b7773b6b257d5eefa8df2df43ac4e","author":{"name":"truckman","email":"7db9b6fb22bf58dcc1fd7606fbae5570b479b7be@FreeBSD.org"},"message":"MFC r300633\n\nFix acpidb CIDs 1011279 (Buffer not null terminated) and 978405 and\n1199380 (Resource leak).\n\nload_dsdt() calls strncpy() to copy a filename and Coverity warns\nthat the destination buffer may not be NUL terminated.  Fix this\nby using strlcpy() instead.  If silent truncation occurs, then the\nfilename was not valid anyway.\n\nload_dsdt() leaks an fd (CID 978405) and a memory region allocated\nusing mmap() (CID 1199380) when it returns.  Fix these by calling\nclose() and munmap() as appropriate.\n\nDon't bother fixing the minor memory leak \"list\", allocated by\nAcGetAllTablesFromFile() (CID 1355191).\n\nCheck for truncation when creating the temp file name.\n\nSet a flag to indicate that the temp file should be unlinked.\nRelying on a strcmp() test could delete the input file in contrived\ncases.\n\nReported by:\tCoverity\nCID:\t\t1011279, 978405, 1199380\nReviewed by:\tjkim\nDifferential Revision:\thttps://reviews.freebsd.org/D6368","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/a88b1b7c540b7773b6b257d5eefa8df2df43ac4e"},{"sha":"3b8f7769ef53354f14c7d2f559208e48bc5fa6fd","author":{"name":"truckman","email":"7db9b6fb22bf58dcc1fd7606fbae5570b479b7be@FreeBSD.org"},"message":"MFC r300635\n\nCall free(cur) if strdup(dirp->d_name) fails to avoid a memory leak.\n\nReported by:\tCoverity\nCID:\t\t1007414","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/3b8f7769ef53354f14c7d2f559208e48bc5fa6fd"},{"sha":"cc25dba923e2f1dcea656ddec203bcd20a6d75e0","author":{"name":"truckman","email":"7db9b6fb22bf58dcc1fd7606fbae5570b479b7be@FreeBSD.org"},"message":"MFC r300636\n\nDelay calling yp_malloc_dnsent() until after some additional sanity\nchecks to avoid leaking memory on error returns.\n\nReported by:\tCoverity\nCID:\t\t1007416","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/cc25dba923e2f1dcea656ddec203bcd20a6d75e0"},{"sha":"0d4a430fb149a2d2e01ee4d837445f358c02a598","author":{"name":"asomers","email":"98dace73f9f4aeb631994692d06377058a93a391@FreeBSD.org"},"message":"MFC r298483, r298744, r299085, r299087, and r300747 to usr.sbin/extattr\n\nr300747 | asomers | 2016-05-26 08:26:49 -0600 (Thu, 26 May 2016) | 8 lines\n\nrmextattr(8) man page clarifications regarding -qq\n\nr299087 | asomers | 2016-05-04 15:07:30 -0600 (Wed, 04 May 2016) | 13 lines\n\nFix \"getextattr -x\" with non-ascii attribute values\n\nextattr/rmextattr.c\n        When printing hex output, treat all attribute values as unsigned\n        char arrays instead of sign extending them to 32 bit values.\n\nextattr/tests/extattr_test.sh\n        Add a regression test\n\nr299085 | asomers | 2016-05-04 14:20:55 -0600 (Wed, 04 May 2016) | 30 lines\n\nAllow setextattr(8) to take attribute values from stdin\n\nAdd the -i option to setextattr. This option allow extended attribute data\nto be provided via stdin. Add a -qq option to getextattr, which omits the\ntrailing newline. Together these options can be used to work with extended\nattributes whose values are large and/or binary.\n\nusr.sbin/extattr/Makefile:\n        Link against libsbuf which is used for processing stdin data.\n\nusr.sbin/extattr/rmextattr.8:\n        Document setextattr's -i option, getextattr's -qq option, and remove\n        the BUG about setextattr only being useful for strings.\n\nusr.sbin/extattr/rmextattr.c:\n        For setextattr operations, buffer attribute data in an sbuf. If -i\n        is specified, pull the data from stdin, otherwise from the\n        appropriate argurment.\n\n        Update usage text and argument validation code for setextattr's -i\n        option.\n\nusr.sbin/extattr/tests/extattr_test.sh\n        Add tests for -q and -i.\n\nr298744 | asomers | 2016-04-28 09:13:50 -0600 (Thu, 28 Apr 2016) | 11 lines\n\nAdd PACKAGE fields to usr.sbin/extattr/tests/Makefile\n\nusr.sbin/extattr/tests/Makefile\n        Add boiler plate required by 298107 but omitted by 298483.  These\n        two changes passed through CR in parallel.  I think this should get\n        the full test suite running in Jenkins again.\n\nr298483 | asomers | 2016-04-22 11:02:47 -0600 (Fri, 22 Apr 2016) | 10 lines\n\nAdd ATF tests for usr.sbin/extattr\n\nAdd ATF tests for the existing behavior of setextattr, rmextattr, lsextattr,\nand getextattr.\n\nSponsored by:\tSpectra Logic Corp","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/0d4a430fb149a2d2e01ee4d837445f358c02a598"},{"sha":"10d7f27879527fd48fe5906f0141d54d5b2c69aa","author":{"name":"truckman","email":"7db9b6fb22bf58dcc1fd7606fbae5570b479b7be@FreeBSD.org"},"message":"MFC r300642\n\nMatch the descriptions of the \\H and \\h prompt string sequences to reality.\nThey were swapped.\n\nX-Confirmed by:\tjilles","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/10d7f27879527fd48fe5906f0141d54d5b2c69aa"},{"sha":"ed265b2890fdcb1fa5b9bbdaddfd54fb5e3cf2ed","author":{"name":"truckman","email":"7db9b6fb22bf58dcc1fd7606fbae5570b479b7be@FreeBSD.org"},"message":"MFC r300643\n\nClose from_fd if malloc() fails to avoid a file descriptor leak.\n\nReported by:\tCoverity\nCID:\t\t1007203","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/ed265b2890fdcb1fa5b9bbdaddfd54fb5e3cf2ed"},{"sha":"efd2996ff9ab633d8be09fa755ee1b021937fede","author":{"name":"truckman","email":"7db9b6fb22bf58dcc1fd7606fbae5570b479b7be@FreeBSD.org"},"message":"MFC r300648\n\nFix CID 1011370 (Resource leak) in ps.\n\nThere is no need to to call strdup() on the value returned by fmt().\nThe latter calls fmt_argv() which always returns a dynamically\nallocated string, and calling strdup() on that leaks the memory\nallocated by fmt_argv().  Wave some const magic on ki_args and\nki_env to make the direct assignment happy.  This requires a tweak\nto the asprintf() case to avoid a const vs. non-const mismatch.\n\nReported by:\tCoverity\nCID:\t\t1011370","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/efd2996ff9ab633d8be09fa755ee1b021937fede"},{"sha":"281ac18e42f4f5ca01787dd3b30e1af7c09b8f80","author":{"name":"truckman","email":"7db9b6fb22bf58dcc1fd7606fbae5570b479b7be@FreeBSD.org"},"message":"MFC r300649\n\nFix Coverity CID 1019054 (String not null terminated) in setfacl.\n\nIncrease the size of buf[] by one to allow room for a NUL character\nat the end.\n\nReported by:\tCoverity\nCID:\t\t1019054","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/281ac18e42f4f5ca01787dd3b30e1af7c09b8f80"},{"sha":"c8f45181287c0cd144f51bd035b3dd26f633a2b2","author":{"name":"truckman","email":"7db9b6fb22bf58dcc1fd7606fbae5570b479b7be@FreeBSD.org"},"message":"MFC r300660\n\nFix 1016718 Resource leak.\n\nDon't leak a file descriptor if fchdir() fails.\n\nReported by:\tCoverity\nCID:\t\t1016718","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/c8f45181287c0cd144f51bd035b3dd26f633a2b2"},{"sha":"fa7d0f23a5adc1d2f858496ce03e3c066f8a0e91","author":{"name":"truckman","email":"7db9b6fb22bf58dcc1fd7606fbae5570b479b7be@FreeBSD.org"},"message":"MFC r300662\n\nFix Coverity CID 1016714 Resource leak in process_file_actions_entry()\n\nDon't leak a file descriptor of _dup2() fails (shouldn't happen).\n\nReported by:\tCoverity\nCID:\t\t1016714","distinct":true,"url":"https://api.github.com/repos/trueos/trueos/commits/fa7d0f23a5adc1d2f858496ce03e3c066f8a0e91"}]}
{"actor":{"display_login":"jordanhubbard"}}