{"push_id":1129127568,"size":54,"distinct_size":54,"ref":"refs/heads/user/landonf/bhnd/1012_pmu_bringup","head":"dbafe7c6903a0b03566005f5f7681418bd3c01cc","before":"93b518a99872b1dac440a3d0876bf404cdc48fa7","commits":[{"sha":"d79d0c45f4795d005d21524a76b5455f90b8e871","author":{"email":"267970d0842910b9668108d47022a729a30ec04b@FreeBSD.org","name":"bdrewery"},"message":"WITH_META_MODE: Correct description of missing .meta file.\n\nSponsored by:\tEMC / Isilon Storage Division","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/d79d0c45f4795d005d21524a76b5455f90b8e871"},{"sha":"8d9f7ad55199db2d543169880be2772bde86fe18","author":{"email":"267970d0842910b9668108d47022a729a30ec04b@FreeBSD.org","name":"bdrewery"},"message":"Regenerate","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/8d9f7ad55199db2d543169880be2772bde86fe18"},{"sha":"4b5433e33a1d4f5587905894fb1c76a5f99a49d4","author":{"email":"61d9507495c15110bcab60505477c9cfeb576ff9@FreeBSD.org","name":"truckman"},"message":"Fix 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\nMFC after:\t1 week\nDifferential Revision:\thttps://reviews.freebsd.org/D6360","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/4b5433e33a1d4f5587905894fb1c76a5f99a49d4"},{"sha":"b29064e902d9e5a98dcbbbeffb19e29bb5c62834","author":{"email":"61d9507495c15110bcab60505477c9cfeb576ff9@FreeBSD.org","name":"truckman"},"message":"Fix 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\nMFC after:\t1 week\nDifferential Revision:\thttps://reviews.freebsd.org/D6368","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/b29064e902d9e5a98dcbbbeffb19e29bb5c62834"},{"sha":"b6a7cc7788ab1bad5330f783ba585e9148772859","author":{"email":"baa3cdac49150ef6689c8016eab0ce257d183a8f@FreeBSD.org","name":"cem"},"message":"efi loader: Match format string to EFI_ERROR_CODE()\n\nSilence a format specifier warning.\n\nSponsored by:\tEMC / Isilon Storage Division","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/b6a7cc7788ab1bad5330f783ba585e9148772859"},{"sha":"83a68a204841e8eb8fc6cdfe32fd5249b8be8358","author":{"email":"61d9507495c15110bcab60505477c9cfeb576ff9@FreeBSD.org","name":"truckman"},"message":"Call free(cur) if strdup(dirp->d_name) fails to avoid a memory leak.\n\nReported by:\tCoverity\nCID:\t\t1007414\nMFC after:\t1 week","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/83a68a204841e8eb8fc6cdfe32fd5249b8be8358"},{"sha":"0eaef17c3b4f468a3eb170aa0b7b6563d24cb177","author":{"email":"61d9507495c15110bcab60505477c9cfeb576ff9@FreeBSD.org","name":"truckman"},"message":"Delay calling yp_malloc_dnsent() until after some additional sanity\nchecks to avoid leaking memory on error returns.\n\nReported by:\tCoverity\nCID:\t\t1007416\nMFC after:\t1 week","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/0eaef17c3b4f468a3eb170aa0b7b6563d24cb177"},{"sha":"1733b7974052a8bcdb179287323e2027ba161c70","author":{"email":"3bae1b7d571c203acfb4d62748e2c04607e018fd@FreeBSD.org","name":"jhibbits"},"message":"Set the TLB caching properties for portals at attach time.\n\nThis was found while reworking the device tree nodes for dtsec to match the\nLinux device tree.  Instead of waiting and expecting later code to call\ndpaa_portal_map_registers(), do the equivalent immediately upon mapping.\nOtherwise, it's possible to access the pages before that function is called, and\nhang the CPU.","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/1733b7974052a8bcdb179287323e2027ba161c70"},{"sha":"e825b90448b365da566c7370cec6f3e13bd88697","author":{"email":"31ace4ad1831aae866cd7951a842ca3e38f21981@FreeBSD.org","name":"cy"},"message":"Use the expiry date to determine whether to replace the DB copy of\nleapfile instead of using the leapfile serial number (create\ntimestamp).\n\nPR:\t\t209577\nMFC after:\t3 days","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/e825b90448b365da566c7370cec6f3e13bd88697"},{"sha":"610c8f37edec3884d4c6ff72b64b78bac7a3166a","author":{"email":"61d9507495c15110bcab60505477c9cfeb576ff9@FreeBSD.org","name":"truckman"},"message":"Fix Coverity CIDs 1340544 Resource leak and 1340543 Use after free\n\nAt line 479 of ldapclient.c in client_build_req(), the error return\nleaks ldap_attrs (CID 1340544).  It looks like this can happen if\nthe first utoa() call in aldap_get_stringset() fails.  It looks\nlike other leaks can happen if other utoa() calls fail since scanning\nthis array when it is freed stops when the first NULL is encountered.\nFix these problems by not storing NULL in the array when utoa()\nfails, and by freeing ret and returning NULL if nothing is stored\nin the array.  That way the caller will never see the\nldap_attrs[0] == NULL case, so delete that check.\n\nThe ber_printf_element() calls ber_free_elements() on its ber\nargument and returns NULL on failure.  When each of its callers\ndetects failure, they do a goto fail, which then calls ber_free_elements()\nwith the same pointer (CID 1340543).  Fix is to delete the\nber_free_elements() from ber_printf_element()\n\nReported by:\tCoverity\nCID:\t\t1340543, 1340544\nReviewed by:\taraujo\nDifferential Revision:\thttps://reviews.freebsd.org/D6550","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/610c8f37edec3884d4c6ff72b64b78bac7a3166a"},{"sha":"896f16ef24bdfb6b72b4c91ad32314c8eb970b6d","author":{"email":"470bc578162732ac7f9d387d34c4af4ca6e1b6f7@FreeBSD.org","name":"ken"},"message":"Fix a few ada(4) driver issues:\n\n o Some Samsung drives do not support the ATA READ LOG EXT or READ\n   LOG DMA EXT commands, despite indicating that they do in their\n   IDENTIFY data.  So, fix this in two ways:\n\t1. Only start the log directory probe (ADA_STATE_LOGDIR) if\n\t   the drive claims to be an SMR drive in the first place.\n\t   We don't need to do the extra probing for other devices.\n\t   This will also serve to prevent problems with other\n\t   drives that have the same issue.\n\t2. Add quirks for the two Samsung drives that have been\n\t   reported so far (thanks to Oleg Nauman and Alex Petrov).\n\t   If there is a reason to do a Read Log later on, we will\n\t   know that it doesn't work on these drives.\n\n o Add a quirk entry to mark Seagate Lamarr Drive Managed drives as\n   drive managed.  They don't report this in their Identify data.\n\nsys/cam/ata/ata_da.c:\n\tAdd two new quirks:\n\t1. ADA_Q_LOG_BROKEN, for drives that claim to support Read\n\t   Log but don't really.\n\t2. ADA_Q_SMR_DM, for drives that are Drive Managed SMR, but\n\t   don't report it.  This can matter for software that\n\t   wants to know when it should make an extra effort to\n\t   write sequentially.\n\n\tRecord two Samsung drives that don't support Read Log, and\n\tone Seagate drive that doesn't report that it is a SMR drive.\n\tThe Seagate drive is already recorded in the da(4) driver.\n\n\tWe may have to come up with a similar solution in the da(4)\n\tdriver for SATA drives that don't properly support Read Log.\n\n\tIn adasetflags(), Dont' set the ADA_FLAG_CAN_LOG bit if the\n\tdevice has the LOG_BROKEN quirk set.  Also, look at the\n\tSMR_DM quirk and set the device type accordingly if it is\n\tactually a drive managed drive.\n\n\tWhen deciding whether to go into the LOGDIR probe state,\n\tlook to see whether the device claims to be an SMR device.\n\tIf not, don't bother with the LOGDIR probe state.\n\nSponsored by:\tSpectra Logic","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/896f16ef24bdfb6b72b4c91ad32314c8eb970b6d"},{"sha":"7fa8459fdb5367b97611bd13ebda4810a9261f4e","author":{"email":"61d9507495c15110bcab60505477c9cfeb576ff9@FreeBSD.org","name":"truckman"},"message":"Match the descriptions of the \\H and \\h prompt string sequences to reality.\nThey were swapped.\n\nX-Confirmed by:\tjilles\nMFC after:\t1 week","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/7fa8459fdb5367b97611bd13ebda4810a9261f4e"},{"sha":"36fa61883428b677cc1cbf35c851cf4dbbf4990b","author":{"email":"61d9507495c15110bcab60505477c9cfeb576ff9@FreeBSD.org","name":"truckman"},"message":"Close from_fd if malloc() fails to avoid a file descriptor leak.\n\nReported by:\tCoverity\nCID:\t\t1007203\nMFC after:\t1 week","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/36fa61883428b677cc1cbf35c851cf4dbbf4990b"},{"sha":"d75fddcdf511f78cdb064b17d0740fb7959b77cd","author":{"email":"153549eac10244a41dca49732ef831d5104286aa@FreeBSD.org","name":"sephe"},"message":"hyperv/vmbus: Check hyperv_dmamem_alloc return value\n\nThough it is highly unlikely this function would fail w/ BUS_DMA_WAITOK,\nwe had better to check its return value; better safe then sorry here.\n\nMFC after:\t1 week\nSponsored by:\tMicrosoft OSTC\nDifferential Revision:\thttps://reviews.freebsd.org/D6518","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/d75fddcdf511f78cdb064b17d0740fb7959b77cd"},{"sha":"7379c4d052bbec13b7aee401e38fd50d18a412d7","author":{"email":"153549eac10244a41dca49732ef831d5104286aa@FreeBSD.org","name":"sephe"},"message":"hyperv/vmbus: Allocate/setup IDT vector after all ISR resources are ready\n\nAnd release IDT vector before releasing ISR resources on interrupt\nteardown path.  We still have some work to do on the interrupt tearing\ndown path.\n\nMFC after:\t1 week\nSponsored by:\tMicrosoft OSTC\nDifferential Revision:\thttps://reviews.freebsd.org/D6519","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/7379c4d052bbec13b7aee401e38fd50d18a412d7"},{"sha":"b29320f7e9b593852061315dfdf7aeb3b700f573","author":{"email":"153549eac10244a41dca49732ef831d5104286aa@FreeBSD.org","name":"sephe"},"message":"hyperv/vmbus: Move event/message taskqueue/task to vmbus softc\n\nMFC after:\t1 week\nSponsored by:\tMicrosoft OSTC\nDifferential Revision:\thttps://reviews.freebsd.org/D6520","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/b29320f7e9b593852061315dfdf7aeb3b700f573"},{"sha":"2a3e3576d0fa51c3ead7a13253e68e84849a1bf9","author":{"email":"153549eac10244a41dca49732ef831d5104286aa@FreeBSD.org","name":"sephe"},"message":"hyperv/vmbus: Move vcpuid into vmbus softc per-cpu data\n\nMFC after:\t1 week\nSponsored by:\tMicrosoft OSTC\nDifferential Revision:\thttps://reviews.freebsd.org/D6521","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/2a3e3576d0fa51c3ead7a13253e68e84849a1bf9"},{"sha":"01033af8a17669d80647004092e350d756fef029","author":{"email":"61d9507495c15110bcab60505477c9cfeb576ff9@FreeBSD.org","name":"truckman"},"message":"Fix 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\nMFC after:\t1 week","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/01033af8a17669d80647004092e350d756fef029"},{"sha":"e891aa6cfef5b58a83fd46c7e9b4ffc9b053157e","author":{"email":"61d9507495c15110bcab60505477c9cfeb576ff9@FreeBSD.org","name":"truckman"},"message":"Fix 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\nMFC after:\t1 week","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/e891aa6cfef5b58a83fd46c7e9b4ffc9b053157e"},{"sha":"ae7646a74b6dfec8dd3ed52935b79c39ca2b55fd","author":{"email":"153549eac10244a41dca49732ef831d5104286aa@FreeBSD.org","name":"sephe"},"message":"hyperv/vmbus: Move two global flags into vmbus softc\n\nAnd pack them into one flag field.\n\nMFC after:\t1 week\nSponsored by:\tMicrosoft OSTC\nDifferential Revision:\thttps://reviews.freebsd.org/D6522","distinct":true,"url":"https://api.github.com/repos/landonf/freebsd/commits/ae7646a74b6dfec8dd3ed52935b79c39ca2b55fd"}]}
{"actor":{"display_login":"landonf"}}