qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] d0e31a: Remove reduntant qemu: from error fun


From: GitHub
Subject: [Qemu-commits] [qemu/qemu] d0e31a: Remove reduntant qemu: from error functions
Date: Thu, 11 May 2017 13:07:18 -0700

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: d0e31a105e2a353a2ac148c8f0044cfa8939f645
      
https://github.com/qemu/qemu/commit/d0e31a105e2a353a2ac148c8f0044cfa8939f645
  Author: Ishani Chugh <address@hidden>
  Date:   2017-05-07 (Sun, 07 May 2017)

  Changed paths:
    M hw/microblaze/boot.c
    M hw/nios2/boot.c
    M hw/ppc/pnv.c
    M hw/s390x/sclp.c
    M hw/tricore/tricore_testboard.c
    M numa.c

  Log Message:
  -----------
  Remove reduntant qemu: from error functions

This patch removes redundant "qemu:" from error functions. The link to the 
bitesized task is:
http://wiki.qemu-project.org/Contribute/BiteSizedTasks#Error_checking

Signed-off-by: Ishani Chugh <address@hidden>
Reviewed-by: Eduardo Habkost <address@hidden>
Reviewed-by: David Gibson <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 2d812d6dff5267d80c54125f15a090539573d29e
      
https://github.com/qemu/qemu/commit/2d812d6dff5267d80c54125f15a090539573d29e
  Author: KONRAD Frederic <address@hidden>
  Date:   2017-05-07 (Sun, 07 May 2017)

  Changed paths:
    M hw/ppc/ppc_booke.c

  Log Message:
  -----------
  ppc_booke: drop useless assignment

The tb_env variable is set two lines above. So just drop the double assignment.

Signed-off-by: KONRAD Frederic <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 61f7c6a0c200db3749f131ff023fa50e7f71814e
      
https://github.com/qemu/qemu/commit/61f7c6a0c200db3749f131ff023fa50e7f71814e
  Author: Marc-André Lureau <address@hidden>
  Date:   2017-05-07 (Sun, 07 May 2017)

  Changed paths:
    M include/io/channel.h

  Log Message:
  -----------
  doc: fix function spelling

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: ed1fcd00094cdd9395315068442abe28f612e8a3
      
https://github.com/qemu/qemu/commit/ed1fcd00094cdd9395315068442abe28f612e8a3
  Author: Craig Jellick <address@hidden>
  Date:   2017-05-07 (Sun, 07 May 2017)

  Changed paths:
    M qemu-options.hx

  Log Message:
  -----------
  Add 'none' as type for drive's if option

Signed-off-by: Craig Jellick <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 46bbbec2d30c3d791a6d328e5ef861694ff3eafb
      
https://github.com/qemu/qemu/commit/46bbbec2d30c3d791a6d328e5ef861694ff3eafb
  Author: Eric Blake <address@hidden>
  Date:   2017-05-07 (Sun, 07 May 2017)

  Changed paths:
    M tests/.gitignore

  Log Message:
  -----------
  tests: Ignore more test executables

Ignore test executables when building in-tree:
test-arm-mptimer introduced in commit 882fac3
test-crypto-hmac introduced in commit 4fd460b
test-aio-multithread introduced in commit 0c330a7

Signed-off-by: Eric Blake <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 6b1de1484ee2f80d5795358ea79e90c3ceb64167
      
https://github.com/qemu/qemu/commit/6b1de1484ee2f80d5795358ea79e90c3ceb64167
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2017-05-07 (Sun, 07 May 2017)

  Changed paths:
    M hw/usb/dev-smartcard-reader.c

  Log Message:
  -----------
  usb-ccid: make ccid_write_data_block() cope with null buffers

static code analyzer complain:

hw/usb/dev-smartcard-reader.c:816:5: warning: Null pointer passed as an 
argument to a 'nonnull' parameter
    memcpy(p->abData, data, len);
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~

Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 21a9ad2f159e4cafd090cc2b47ce322e024c7363
      
https://github.com/qemu/qemu/commit/21a9ad2f159e4cafd090cc2b47ce322e024c7363
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2017-05-07 (Sun, 07 May 2017)

  Changed paths:
    M device_tree.c

  Log Message:
  -----------
  device_tree: fix compiler warnings (clang 5)

static code analyzer complain:

device_tree.c:155:18: warning: Null pointer passed as an argument to a 
'nonnull' parameter
    while ((de = readdir(d)) != NULL) {
           ^~~~~~~~~~

Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 9879f5ac62c1e0c92a5ff2f4a0ada1b66d43da90
      
https://github.com/qemu/qemu/commit/9879f5ac62c1e0c92a5ff2f4a0ada1b66d43da90
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2017-05-07 (Sun, 07 May 2017)

  Changed paths:
    M qga/commands-posix.c

  Log Message:
  -----------
  qga: fix compiler warnings (clang 5)

static code analyzer complain:

qga/commands-posix.c:2127:9: warning: Null pointer passed as an argument to a 
'nonnull' parameter
  closedir(dp);
  ^~~~~~~~~~~~

Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>
Reviewed-by: Michael Roth <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: ec45bbe5f1921c6553fbf9c0c76b358b0403c22d
      
https://github.com/qemu/qemu/commit/ec45bbe5f1921c6553fbf9c0c76b358b0403c22d
  Author: Saurav Sachidanand <address@hidden>
  Date:   2017-05-07 (Sun, 07 May 2017)

  Changed paths:
    M bsd-user/main.c
    M linux-user/main.c
    M util/envlist.c

  Log Message:
  -----------
  util: Use g_malloc/g_free in envlist.c

Change malloc/strdup/free to g_malloc/g_strdup/g_free in
util/envlist.c.

Remove NULL checks for pointers returned from g_malloc and g_strdup
as they exit in case of failure. Also, update calls to envlist_create
to reflect this.

Free array and array contents returned by envlist_to_environ using
g_free in bsd-user/main.c and linux-user/main.c.

Update comments to reflect change in semantics.

Signed-off-by: Saurav Sachidanand <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 3c76c606dab4d2298a9877628fe28bf9e6d99e22
      
https://github.com/qemu/qemu/commit/3c76c606dab4d2298a9877628fe28bf9e6d99e22
  Author: Fam Zheng <address@hidden>
  Date:   2017-05-07 (Sun, 07 May 2017)

  Changed paths:
    M block/replication.c

  Log Message:
  -----------
  block: Make 'replication_state' an enum

BDRVReplicationState.replication_state is a name with a bit of
duplication, plus it could be an enum like BDRVReplicationState.mode,
which is more readable and also more straightforward in a debugger.

Rename it, and improve the type while at it.

Signed-off-by: Fam Zheng <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: c6a9a9f57503a2736c08711a0387c3e7718353ba
      
https://github.com/qemu/qemu/commit/c6a9a9f57503a2736c08711a0387c3e7718353ba
  Author: Daniel P. Berrange <address@hidden>
  Date:   2017-05-09 (Tue, 09 May 2017)

  Changed paths:
    M qemu-doc.texi
    M qemu.sasl

  Log Message:
  -----------
  Default to GSSAPI (Kerberos) instead of DIGEST-MD5 for SASL

RFC 6331 documents a number of serious security weaknesses in
the SASL DIGEST-MD5 mechanism. As such, QEMU should not be
using or recommending it as a default mechanism for VNC auth
with SASL.

GSSAPI (Kerberos) is the only other viable SASL mechanism that
can provide secure session encryption so enable that by defalt
as the replacement. If users have TLS enabled for VNC, they can
optionally decide to use SCRAM-SHA-1 instead of GSSAPI, allowing
plain username and password auth.

Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>


  Commit: 899833cd65cafbdbae4b5b9da3b1c3726e13e299
      
https://github.com/qemu/qemu/commit/899833cd65cafbdbae4b5b9da3b1c3726e13e299
  Author: Daniel P. Berrange <address@hidden>
  Date:   2017-05-09 (Tue, 09 May 2017)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  List SASL config file under the cryptography maintainer's realm

No one is listed as maintainer for qemu.sasl. It is used by the
VNC server for SASL auth, but since it is cryptography related,
list it under the crytography maintainer's realm, rather than
under the UI maintainer.

Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>


  Commit: e4a3507e86a1ef1453d603031bca27d5ac4cff3c
      
https://github.com/qemu/qemu/commit/e4a3507e86a1ef1453d603031bca27d5ac4cff3c
  Author: Daniel P. Berrange <address@hidden>
  Date:   2017-05-09 (Tue, 09 May 2017)

  Changed paths:
    M block/crypto.c
    M crypto/block-luks.c
    M include/crypto/block.h
    M tests/test-crypto-block.c

  Log Message:
  -----------
  crypto: move 'opaque' parameter to (nearly) the end of parameter list

Previous commit moved 'opaque' to be the 2nd parameter in the list:

  commit 375092332eeaa6e47561ce47fd36144cdaf964d0
  Author: Fam Zheng <address@hidden>
  Date:   Fri Apr 21 20:27:02 2017 +0800

    crypto: Make errp the last parameter of functions

    Move opaque to 2nd instead of the 2nd to last, so that compilers help
    check with the conversion.

this puts it back to the 2nd to last position.

Reviewed-by: Eric Blake <address@hidden>
Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>


  Commit: a37278169d0d7d33270c6817f4a2e4f7345e1098
      
https://github.com/qemu/qemu/commit/a37278169d0d7d33270c6817f4a2e4f7345e1098
  Author: Geert Martin Ijewski <address@hidden>
  Date:   2017-05-09 (Tue, 09 May 2017)

  Changed paths:
    M crypto/init.c
    M crypto/random-gcrypt.c
    M crypto/random-gnutls.c
    M crypto/random-platform.c
    M include/crypto/random.h

  Log Message:
  -----------
  crypto: qcrypto_random_bytes() now works on windows w/o any other crypto libs

If no crypto library is included in the build, QEMU uses
qcrypto_random_bytes() to generate random data. That function tried to open
/dev/urandom or /dev/random and if opening both files failed it errored out.

Those files obviously do not exist on windows, so there the code uses
CryptGenRandom().

Furthermore there was some refactoring and a new function
qcrypto_random_init() was introduced. If a proper crypto library (gnutls or
libgcrypt) is included in the build, this function does nothing. If neither
is included it initializes the (platform specific) handles that are used by
qcrypto_random_bytes().
Either:
* a handle to /dev/urandom | /dev/random on unix like systems
* a handle to a cryptographic service provider on windows

Signed-off-by: Geert Martin Ijewski <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>


  Commit: bcd711feb0420d66e069498ed243a9e2b81ec167
      
https://github.com/qemu/qemu/commit/bcd711feb0420d66e069498ed243a9e2b81ec167
  Author: sochin.jiang <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M include/io/channel-file.h

  Log Message:
  -----------
  channel-file: fix wrong parameter comments

Signed-off-by: sochin.jiang <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 09d352042fa91d296079e449c4d7fd6f42d5db05
      
https://github.com/qemu/qemu/commit/09d352042fa91d296079e449c4d7fd6f42d5db05
  Author: Andreas Grapentin <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M configure
    M include/qemu/compiler.h

  Log Message:
  -----------
  use _Static_assert in QEMU_BUILD_BUG_ON

QEMU_BUILD_BUG_ON should use C11's _Static_assert, if the compiler supports it,
to provide more readable messages on failure.

We check for _Static_assert in configure, and set CONFIG_STATIC_ASSERT
accordingly. QEMU_BUILD_BUG_ON invokes _Static_assert if CONFIG_STATIC_ASSERT
is defined, and reverts to the old way otherwise.

That way, systems without C11 conforming compiler will still have the old
messages, as verified by intentionally breaking the configure check.

the following example output was generated by inverting the condition in
QEMU_BUILD_BUG_ON:

without _Static_assert:

> In file included from /qemu/include/qemu/osdep.h:36:0,
>                  from /qemu/qga/commands.c:13:
> /qemu/qga/commands.c: In function ‘qmp_guest_exec_status’:
> /qemu/include/qemu/compiler.h:89:12: error: negative width in bit-field 
> ‘<anonymous>’
>      struct { \
>             ^
> /qemu/include/qemu/compiler.h:96:38: note: in expansion of macro  
> QEMU_BUILD_BUG_ON_STRUCT’
>  #define QEMU_BUILD_BUG_ON(x) typedef QEMU_BUILD_BUG_ON_STRUCT(x) \
>                                       ^~~~~~~~~~~~~~~~~~~~~~~~
> /qemu/include/qemu/atomic.h:146:5: note: in expansion of macro 
> ‘QEMU_BUILD_BUG_ON’
>      QEMU_BUILD_BUG_ON(sizeof(*ptr) > sizeof(void *));   \
>      ^~~~~~~~~~~~~~~~~
> /qemu/include/qemu/atomic.h:417:5: note: in expansion of macro 
> ‘atomic_load_acquire’
>      atomic_load_acquire(ptr)
>      ^~~~~~~~~~~~~~~~~~~
> /qemu/qga/commands.c:160:21: note: in expansion of macro ‘atomic_mb_read’
>      bool finished = atomic_mb_read(&gei->finished);
>                      ^~~~~~~~~~~~~~

with _Static_assert:

> In file included from /qemu/include/qemu/osdep.h:36:0,
>                  from /qemu/qga/commands.c:13:
> /qemu/qga/commands.c: In function ‘qmp_guest_exec_status’:
> /qemu/include/qemu/compiler.h:94:30: error: static assertion failed: "not 
> expecting: sizeof(*&gei->finished) > sizeof(void *)"
>  #define QEMU_BUILD_BUG_ON(x) _Static_assert(!(x), #x)
>                               ^
> /qemu/include/qemu/atomic.h:146:5: note: in expansion of macro 
> ‘QEMU_BUILD_BUG_ON’
>      QEMU_BUILD_BUG_ON(sizeof(*ptr) > sizeof(void *));   \
>      ^~~~~~~~~~~~~~~~~
> /qemu/include/qemu/atomic.h:417:5: note: in expansion of macro 
> ‘atomic_load_acquire’
>      atomic_load_acquire(ptr)
>      ^~~~~~~~~~~~~~~~~~~
> /qemu/qga/commands.c:160:21: note: in expansion of macro ‘atomic_mb_read’
>      bool finished = atomic_mb_read(&gei->finished);
>                      ^~~~~~~~~~~~~~

Signed-off-by: Andreas Grapentin <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 3ba34a70223a78e89094c7f0a2f7cb6a667b8f9e
      
https://github.com/qemu/qemu/commit/3ba34a70223a78e89094c7f0a2f7cb6a667b8f9e
  Author: Thomas Huth <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M qemu-doc.texi

  Log Message:
  -----------
  qemu-doc: Fix broken URLs of amnhltm.zip and dosidle210.zip

There are some broken URLs in the qemu-doc which reference tools that
are not available at their original location anymore. Fortunately, they
have been mirrored to archive.org, so point to that location instead.

Signed-off-by: Thomas Huth <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 1d29b5b0499c4d1f0415fadf94e41ed8964a7ed3
      
https://github.com/qemu/qemu/commit/1d29b5b0499c4d1f0415fadf94e41ed8964a7ed3
  Author: Fam Zheng <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M hw/block/virtio-blk.c

  Log Message:
  -----------
  virtio-blk: Remove useless condition around g_free()

Laszlo spotted and studied this wasteful "if". He pointed out:

The original virtio_blk_free_request needed an "if" as it accesses one
field, since 671ec3f05655 ("virtio-blk: Convert VirtIOBlockReq.elem to
pointer", 2014-06-11); later on in f897bf751fbd ("virtio-blk: embed
VirtQueueElement in VirtIOBlockReq", 2014-07-09) the field became
embedded, so the "if" became unnecessary (at which point we were using
g_slice_free(), but it is the same.

Now drop it.

Reported-by: Laszlo Ersek <address@hidden>
Signed-off-by: Fam Zheng <address@hidden>
Reviewed-by: Laszlo Ersek <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 6516367fc0803d079384e0ad370856ac328bef30
      
https://github.com/qemu/qemu/commit/6516367fc0803d079384e0ad370856ac328bef30
  Author: Thomas Huth <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M hw/core/generic-loader.c

  Log Message:
  -----------
  hw/core/generic-loader: Fix crash when running without CPU

When running QEMU with "-M none -device loader,file=kernel.elf", it
currently crashes with a segmentation fault, because the "none"-machine
does not have any CPU by default and the generic loader code tries
to dereference s->cpu. Fix it by adding an appropriate check for a
NULL pointer.

Reported-by: Laurent Vivier <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 3baa0a6a658e3a17a79d95440e5c949b2b13bb6e
      
https://github.com/qemu/qemu/commit/3baa0a6a658e3a17a79d95440e5c949b2b13bb6e
  Author: Chris Webb <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M qemu-options.hx
    M vl.c

  Log Message:
  -----------
  virtfs: allow a device id to be specified in the -virtfs option

When using a virtfs root filesystem, the mount_tag needs to be set to
/dev/root. This can be done long-hand as

  -fsdev local,id=root,path=/path/to/rootfs,...
  -device virtio-9p-pci,fsdev=root,mount_tag=/dev/root

but the -virtfs shortcut cannot be used as it hard-codes the device identifier
to match the mount_tag, and device identifiers may not contain '/':

  $ qemu-system-x86_64 -virtfs 
local,path=/foo,mount_tag=/dev/root,security_model=passthrough
  qemu-system-x86_64: -virtfs 
local,path=/foo,mount_tag=/dev/root,security_model=passthrough: duplicate fsdev 
id: /dev/root

To support this case using -virtfs, we allow the device identifier to be
specified explicitly when the mount_tag is not suitable:

  -virtfs local,id=root,path=/path/to/rootfs,mount_tag=/dev/root,...

Signed-off-by: Chris Webb <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 6f75023ab89708101dabc8f710bd0ed501311c8a
      
https://github.com/qemu/qemu/commit/6f75023ab89708101dabc8f710bd0ed501311c8a
  Author: Kamil Rytarowski <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M scripts/qemu-binfmt-conf.sh

  Log Message:
  -----------
  scripts/qemu-binfmt-conf.sh: Fix shell portability issue

Appease pkgsrc and use portable shell variable comparison.
This switches "==" to "=". It should not be a functional change.

Signed-off-by: Kamil Rytarowski <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: b7d5a9c2c672f6088bfa209c1338badbc51de3f3
      
https://github.com/qemu/qemu/commit/b7d5a9c2c672f6088bfa209c1338badbc51de3f3
  Author: Kamil Rytarowski <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M scripts/checkpatch.pl
    M scripts/clean-header-guards.pl
    M scripts/cleanup-trace-events.pl
    M scripts/disas-objdump.pl
    M scripts/get_maintainer.pl
    M scripts/shaderinclude.pl
    M scripts/switch-timer-api
    M scripts/texi2pod.pl

  Log Message:
  -----------
  scripts: Switch to more portable Perl shebang

The default NetBSD package manager is pkgsrc and it installs Perl
along other third party programs under custom and configurable prefix.
The default prefix for binary prebuilt packages is /usr/pkg, and the
Perl executable lands in /usr/pkg/bin/perl.

This change switches "/usr/bin/perl" to "/usr/bin/env perl" as it's
the most portable solution that should work for almost everybody.
Perl's executable is detected automatically.

This change switches -w option passed to the executable with more
modern "use warnings;" approach. There is no functional change to the
default behavior.

Signed-off-by: Kamil Rytarowski <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: fafa2e6702a1ae4f094f8d5b79afef1e641ea3bb
      
https://github.com/qemu/qemu/commit/fafa2e6702a1ae4f094f8d5b79afef1e641ea3bb
  Author: Eric Blake <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M tests/.gitignore

  Log Message:
  -----------
  tests: Ignore another built executable (test-hmp)

Commit 78f86a2b7 added a new test, but forgot to exclude the built
binary from version control.

Signed-off-by: Eric Blake <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: e9c6ab62c760e333a6cf0f3f9ab021633723434c
      
https://github.com/qemu/qemu/commit/e9c6ab62c760e333a6cf0f3f9ab021633723434c
  Author: Paolo Bonzini <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M hw/display/jazz_led.c

  Log Message:
  -----------
  jazz_led: fix bad snprintf

Detected by GCC 7's -Wformat-truncation.  snprintf writes at most
2 bytes here including the terminating NUL, so the result is
truncated.  In addition, the newline at the end is pointless.
Fix the buffer size and the format string.

Signed-off-by: Paolo Bonzini <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 3ecb29a3280b706aebfbc353718c8272eb3ea9db
      
https://github.com/qemu/qemu/commit/3ecb29a3280b706aebfbc353718c8272eb3ea9db
  Author: Paolo Bonzini <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Update paths for main loop

Moved by c2b38b2 ("block: move AioContext, QEMUTimer, main-loop to
libqemuutil"), let's update MAINTAINERS too.

Reported-by: Laurent Vivier <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 36c697bda59acf1f8d67a437403f857ec905f2ba
      
https://github.com/qemu/qemu/commit/36c697bda59acf1f8d67a437403f857ec905f2ba
  Author: Paolo Bonzini <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Update paths for AioContext implementation

Moved by c2b38b2
("block: move AioContext, QEMUTimer, main-loop to libqemuutil")

Signed-off-by: Paolo Bonzini <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: e1ae9fb6c2a35015ed4881def4a4a7b6be2d15f0
      
https://github.com/qemu/qemu/commit/e1ae9fb6c2a35015ed4881def4a4a7b6be2d15f0
  Author: Fam Zheng <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M tests/postcopy-test.c

  Log Message:
  -----------
  tests: Remove redundant assignment

Signed-off-by: Fam Zheng <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 1effe6ad5eac1b2e50a077695ac801d172891d6a
      
https://github.com/qemu/qemu/commit/1effe6ad5eac1b2e50a077695ac801d172891d6a
  Author: Stefan Hajnoczi <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M MAINTAINERS
    M block/crypto.c
    M crypto/block-luks.c
    M crypto/init.c
    M crypto/random-gcrypt.c
    M crypto/random-gnutls.c
    M crypto/random-platform.c
    M include/crypto/block.h
    M include/crypto/random.h
    M qemu-doc.texi
    M qemu.sasl
    M tests/test-crypto-block.c

  Log Message:
  -----------
  Merge remote-tracking branch 'danpb/tags/pull-qcrypto-2017-05-09-1' into 
staging

Merge qcrypto 2017/05/09 v1

# gpg: Signature made Tue 09 May 2017 09:43:47 AM EDT
# gpg:                using RSA key 0xBE86EBB415104FDF
# gpg: Good signature from "Daniel P. Berrange <address@hidden>"
# gpg:                 aka "Daniel P. Berrange <address@hidden>"
# Primary key fingerprint: DAF3 A6FD B26B 6291 2D0E  8E3F BE86 EBB4 1510 4FDF

* danpb/tags/pull-qcrypto-2017-05-09-1:
  crypto: qcrypto_random_bytes() now works on windows w/o any other crypto libs
  crypto: move 'opaque' parameter to (nearly) the end of parameter list
  List SASL config file under the cryptography maintainer's realm
  Default to GSSAPI (Kerberos) instead of DIGEST-MD5 for SASL

Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: f465706e590a6543542246a9f1b591e5be39c568
      
https://github.com/qemu/qemu/commit/f465706e590a6543542246a9f1b591e5be39c568
  Author: Stefan Hajnoczi <address@hidden>
  Date:   2017-05-10 (Wed, 10 May 2017)

  Changed paths:
    M MAINTAINERS
    M block/replication.c
    M bsd-user/main.c
    M device_tree.c
    M hw/block/virtio-blk.c
    M hw/core/generic-loader.c
    M hw/display/jazz_led.c
    M hw/microblaze/boot.c
    M hw/nios2/boot.c
    M hw/ppc/pnv.c
    M hw/ppc/ppc_booke.c
    M hw/s390x/sclp.c
    M hw/tricore/tricore_testboard.c
    M hw/usb/dev-smartcard-reader.c
    M include/io/channel-file.h
    M include/io/channel.h
    M linux-user/main.c
    M numa.c
    M qemu-doc.texi
    M qemu-options.hx
    M qga/commands-posix.c
    M scripts/checkpatch.pl
    M scripts/clean-header-guards.pl
    M scripts/cleanup-trace-events.pl
    M scripts/disas-objdump.pl
    M scripts/get_maintainer.pl
    M scripts/qemu-binfmt-conf.sh
    M scripts/shaderinclude.pl
    M scripts/switch-timer-api
    M scripts/texi2pod.pl
    M tests/.gitignore
    M tests/postcopy-test.c
    M util/envlist.c
    M vl.c

  Log Message:
  -----------
  Merge remote-tracking branch 'mjt/tags/trivial-patches-fetch' into staging

trivial patches for 2017-05-10

# gpg: Signature made Wed 10 May 2017 03:19:30 AM EDT
# gpg:                using RSA key 0x701B4F6B1A693E59
# gpg: Good signature from "Michael Tokarev <address@hidden>"
# gpg:                 aka "Michael Tokarev <address@hidden>"
# gpg:                 aka "Michael Tokarev <address@hidden>"
# Primary key fingerprint: 6EE1 95D1 886E 8FFB 810D  4324 457C E0A0 8044 65C5
#      Subkey fingerprint: 7B73 BAD6 8BE7 A2C2 8931  4B22 701B 4F6B 1A69 3E59

* mjt/tags/trivial-patches-fetch: (23 commits)
  tests: Remove redundant assignment
  MAINTAINERS: Update paths for AioContext implementation
  MAINTAINERS: Update paths for main loop
  jazz_led: fix bad snprintf
  tests: Ignore another built executable (test-hmp)
  scripts: Switch to more portable Perl shebang
  scripts/qemu-binfmt-conf.sh: Fix shell portability issue
  virtfs: allow a device id to be specified in the -virtfs option
  hw/core/generic-loader: Fix crash when running without CPU
  virtio-blk: Remove useless condition around g_free()
  qemu-doc: Fix broken URLs of amnhltm.zip and dosidle210.zip
  use _Static_assert in QEMU_BUILD_BUG_ON
  channel-file: fix wrong parameter comments
  block: Make 'replication_state' an enum
  util: Use g_malloc/g_free in envlist.c
  qga: fix compiler warnings (clang 5)
  device_tree: fix compiler warnings (clang 5)
  usb-ccid: make ccid_write_data_block() cope with null buffers
  tests: Ignore more test executables
  Add 'none' as type for drive's if option
  ...

Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: ecc1f5adeec4e3324d1b695a7c54e3967c526949
      
https://github.com/qemu/qemu/commit/ecc1f5adeec4e3324d1b695a7c54e3967c526949
  Author: Laurent Vivier <address@hidden>
  Date:   2017-05-11 (Thu, 11 May 2017)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  maintainers: Add myself as linux-user reviewer

I volunteer to review linux-user patches.
Adding myself will help to not miss some of them.

Signed-off-by: Laurent Vivier <address@hidden>
Acked-by: Riku Voipio <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>


Compare: https://github.com/qemu/qemu/compare/76d20ea0f1b2...ecc1f5adeec4

reply via email to

[Prev in Thread] Current Thread [Next in Thread]