GitHub PushEvent at 2014-05-28 15:13:21
Repository: qemu
Actor: rth7680
Payload: {"shas":[["78a4b8d2051bff8e8794e9419b7925122212b096","389bd06664b699d49ce291f515b98373073ad5f4@tls.msk.ru","libcacard: g_malloc cleanups\n\nThis patch replaces g_malloc() in libcacard into g_new()\nor g_new0() where appropriate (removing some init-to-zero\nsurrounding code), g_malloc+memcpy into g_memdup() and the\nlike.\n\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>\nReviewed-by: Alon Levy <alevy@redhat.com>","Michael Tokarev",false],["aef553fdcabbea8760cd4647ab14859095300023","313afa09ce093b19ae1dd74f57c1ebd94508584c@linaro.org","iohandler.c: Properly initialize sigaction struct\n\nThe code in qemu_init_child_watch() wasn't clearing the 'struct\nsigaction' before passing it to sigaction(); this meant that we\nwould block a random set of signals while executing the SIGCHLD\nhandler. Initialize properly by using memset() on the struct,\nas we do in similar cases elsewhere.\n\nSigned-off-by: Peter Maydell <peter.maydell@linaro.org>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Peter Maydell",false],["36af599417dde11747a27dc8550ff2281657a8ff","d93fab12d6b381ea21612147809db76a2a401ef5@gmail.com","nbd: Close socket on negotiation failure.\n\nOtherwise, the nbd client may hang waiting for the server response.\n\nSigned-off-by: Hani Benhabiles <hani@linux.com>\nAcked-by: Paolo Bonzini <pbonzini@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Hani Benhabiles",false],["5672ee54d5b52b838e4e6b9976e757f28a68c4ec","d93fab12d6b381ea21612147809db76a2a401ef5@gmail.com","nbd: Miscellaneous typo fixes.\n\nSigned-off-by: Hani Benhabiles <hani@linux.com>\nAcked-by: Paolo Bonzini <pbonzini@redhat.com>\nReviewed-by: Stefan Hajnoczi <stefanha@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Hani Benhabiles",false],["9e04c683fca3da265b1d4f80e1c9cf31bd4f6606","8d4c780fcfdc41841e5070f4c43da8958ba6aec0@weilnetz.de","configure: Automatically select GTK+ 3.0 if GTK+ 2.0 is unavailable\n\nThe configure option --with-gtkabi=3.0 is still supported, but no longer\nneeded when GTK+-2.0 is missing. When no GTK+ ABI is selected by the\nuser, configure first tries 2.0, then 3.0.\n\nFor some platforms (e.g. Windows) newer binaries of GTK+ are only\navailable for GTK+ 3.0. Now building on these platforms is a little bit\neasier.\n\nSigned-off-by: Stefan Weil <sw@weilnetz.de>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Stefan Weil",false],["1a3de8dbecd9a1f628f3e95085158d085725ae52","313afa09ce093b19ae1dd74f57c1ebd94508584c@linaro.org","bswap.h: Rename ldl_p, stl_p, etc to ldl_he_p, stl_he_p, etc\n\nWe have an unfortunate naming clash between the functions\nldl_p, stl_p, etc defined in bswap.h (which have semantics\n\"load/store in host endianness\") and the #defines of the same\nname in cpu-all.h (which have the semantics \"load/store in\ntarget endianness\").\n\nFortunately it turns out that the only users of the bswap.h\nfunctions are all within bswap.h itself, so we can simply\nrename them to include a _he_ infix for \"host endianness\".\n\nSigned-off-by: Peter Maydell <peter.maydell@linaro.org>\nReviewed-by: Richard Henderson <rth@twiddle.net>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Peter Maydell",false],["e35f29ded3602a2ff6776290787c596ce6afc802","d3dad96e79e63fbdaeab67a5628199d2fe088262@gmail.com","jazz_led: Add missing break in switch case\n\nSigned-off-by: Saravanakumar <saravanakumar.punith@gmail.com>\nReviewed-by: Paolo Bonizni <pbonzini@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Saravanakumar",false],["b645000e1ac430601eddb0b435936837aea94bb4","d3dad96e79e63fbdaeab67a5628199d2fe088262@gmail.com","pci: move dereferencing of root only after verifying valid root pointer\n\nSigned-off-by: Saravanakumar <saravanakumar.punith@gmail.com>\nReviewed-by: Andreas Färber <afaerber@suse.de>\nReviewed-by: Paolo Bonzini <pbonzini@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Saravanakumar",false],["0971f1bed21b16e59eb5d15cd03180f110a1e8c0","7d16f0b5dfe3192737120adbafd0f704bf1b8474@gmail.com","arch_init: replace fprintf(stderr, ...) with error_report()\n\nReplace fprintf(stderr,...) with error_report() in the file\narch_init.c. The trailing \"\\n\"s of the @fmt argument have been removed\nbecause @fmt of error_report() should not contain newline.\n\nSigned-off-by: Le Tan <tamlokveer@gmail.com>\nReviewed-by: Andreas Färber <afaerber@suse.de>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Le Tan",false],["9c132c7f64c0605f5c5be79508fa53676be642f9","32041b840daab777010621d827016b7d02d315c5@cs2c.com.cn","dma-helpers: avoid calling dma_bdrv_unmap() twice\n\nCalling dma_bdrv_unmap() twice is not necessary and may cause\npotential problems if some code changes.\n\nSigned-off-by: Jules Wang <junqing.wang@cs2c.com.cn>\nReviewed-by: Paolo Bonzini <pbonzini@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Jules Wang",false],["8cd05ab65a92a592e771a0a1847c7e5505d9a024","313afa09ce093b19ae1dd74f57c1ebd94508584c@linaro.org","configure: Put tempfiles in a subdir of the build directory\n\nWhen libtool support was added to configure, the new temporary files\nwere left out of the list of files cleaned up on exit; this results\nin a lot of stale .lo files being left around in /tmp. Worse, libtool\ncreates a /tmp/.libs directory which we can't easily clean up.\n\nPut all our temporary files in a single temporary directory created\nas a subdirectory of the build directory, so we can easily clean it up,\nand don't need fragile or complicated code for creation to avoid it\nclashing with temporary directories from other instances of QEMU\nconfigure or being subject to attack from adversaries who can write\nto /tmp.\n\nSince the temporaries now live in the build tree, we have no\nneed to jump through hoops with a trap handler to try to remove\nthem when configure exits; this fixes some weird bugs where hitting\n^C during a configure run wouldn't actually make it stop, because\nwe would run the trap handler but then not stop. (It is possible\nto get the trap handler semantics right but it is convoluted largely\nbecause of bugs in dash, so it is simpler to just avoid it.)\n\nNote that \"temporary files go in the build directory, not /tmp\" is\nthe way autoconf behaves.\n\nSigned-off-by: Peter Maydell <peter.maydell@linaro.org>\nReviewed-by: Eric Blake <eblake@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Peter Maydell",false],["5b9d313e3f820f50129887f0a59e88dc376a8b1a","54dff2264884c355ed5584a3b7f1530b4b26921e@redhat.com","vl: fix 'name' option to work with -readconfig\n\nThe 'name' option silently failed when used in config files\n( http://lists.gnu.org/archive/html/qemu-devel/2014-04/msg00378.html )\n\n-readconfig stores the configuration read in QemuOpts.  Command line\noption parsing should do the same, and no more.  In particular it should\nnot act upon the option.  That needs to be done separately, where both\ncommand line and -readconfig settings are visible in QemuOpts.\n\nSigned-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>\nReported-by: William Dauchy <william@gandi.net>\nTested-by: William Dauchy <william@gandi.net>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>\n(mjt: added commit message by ambru@ and subject prefix)","Dr. David Alan Gilbert",false],["d357e3d9d24c8a9e99c994c3c1ee5acb32484a3d","5d00b6b660f20fb41305186c34c40973ec5cb35b@redhat.com","libcacard/vscclient: Bury some dead code\n\nSpotted by Coverity.\n\nSigned-off-by: Markus Armbruster <armbru@redhat.com>\nReviewed-by: Alon Levy <alevy@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Markus Armbruster",false],["124fe7fb1b7a1db8cb2ebb9edae84716ffaf37ce","5d00b6b660f20fb41305186c34c40973ec5cb35b@redhat.com","libcacard: Plug memory leaks around vreader_get_reader_list()\n\nSpotted by Coverity.\n\nSigned-off-by: Markus Armbruster <armbru@redhat.com>\nReviewed-by: Alon Levy <alevy@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Markus Armbruster",false],["fa5912a17b51b807c4ec78a4aa914891c7770781","5d00b6b660f20fb41305186c34c40973ec5cb35b@redhat.com","libcacard/vreader: Drop broken recovery from failed assertion\n\nWe suppress some code when we got unexpected status and assertion\nchecking is off:\n\n     assert(card_status == VCARD_DONE);\n     if (card_status == VCARD_DONE) {\n         int size = MIN(*receive_buf_len, response->b_total_len);\n         memcpy(receive_buf, response->b_data, size);\n         *receive_buf_len = size;\n    }\n\nSuch \"recovery\" is of dubious value even when it works.  This one\ndoesn't: it fails to assign to receive_buf[] and *receive_buf_len,\nwhich the callers expect.\n\nMake the code unconditional.\n\nSigned-off-by: Markus Armbruster <armbru@redhat.com>\nReviewed-by: Alon Levy <alevy@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Markus Armbruster",false],["f33a984d512dbcb6762b002a65de4ea748e31d4a","5d00b6b660f20fb41305186c34c40973ec5cb35b@redhat.com","libcacard/vreader: Tighten assertion to clarify intent\n\nBonus: hushes up Coverity.\n\nSigned-off-by: Markus Armbruster <armbru@redhat.com>\nReviewed-by: Alon Levy <alevy@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Markus Armbruster",false],["26b78f4d3c3f45f84067440e56c3ebf2d25a9574","5d00b6b660f20fb41305186c34c40973ec5cb35b@redhat.com","libcacard: Convert two leftover realloc() to GLib\n\nSigned-off-by: Markus Armbruster <armbru@redhat.com>\nReviewed-by: Alon Levy <alevy@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Markus Armbruster",false],["42119fa3568dc7e8c82447c861678a5987d06d91","5d00b6b660f20fb41305186c34c40973ec5cb35b@redhat.com","libcacard/vcard_emul_nss: Drop a redundant conditional\n\nBailing out when PK11_FindGenericObjects() returns null ensures the\nloop that follows it executes at least once.  The \"loop did not\nexecute\" test right after it is useless.  Drop it.\n\nSpotted by Coverity.\n\nSigned-off-by: Markus Armbruster <armbru@redhat.com>\nReviewed-by: Alon Levy <alevy@redhat.com>\nSigned-off-by: Michael Tokarev <mjt@tls.msk.ru>","Markus Armbruster",false],["479eb12108b66265a8149b0fa90e16900bc217af","aea4fd857e97ee7d33f60175ba9388a955b9d916@twiddle.net","tcg-mips: Layout executable and code_gen_buffer\n\nChoosing good addresses for them means we can use JAL for helper calls.\n\nReviewed-by: Paolo Bonzini <pbonzini@redhat.com>\nSigned-off-by: Richard Henderson <rth@twiddle.net>","Richard Henderson",false],["483c76e14043a60926f7df4415d0e3749694158d","aea4fd857e97ee7d33f60175ba9388a955b9d916@twiddle.net","tcg-mips: Constrain the code_gen_buffer to be within one 256mb segment\n\nThis assures us use of J for exit_tb and goto_tb, and JAL for calling\ninto the generated bswap helpers.\n\nReviewed-by: Paolo Bonzini <pbonzini@redhat.com>\nSigned-off-by: Richard Henderson <rth@twiddle.net>","Richard Henderson",false]],"size":138,"ref":"refs/heads/master","head":"052367ba8573809957f1abd288f79fbbda05a284"}