All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] recipes-devtools: Upgrade QEMU to 3.0
@ 2018-08-15 22:25 Alistair Francis
  2018-08-16  4:05 ` Randy MacLeod
  0 siblings, 1 reply; 4+ messages in thread
From: Alistair Francis @ 2018-08-15 22:25 UTC (permalink / raw)
  To: openembedded-core

Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
---
 meta/conf/distro/include/tcmode-default.inc   |  2 +-
 ...1806-QEMU-slirp-heap-buffer-overflow.patch | 70 -------------------
 .../qemu/{qemu_2.12.0.bb => qemu_3.0.0.bb}    |  7 +-
 3 files changed, 4 insertions(+), 75 deletions(-)
 delete mode 100644 meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch
 rename meta/recipes-devtools/qemu/{qemu_2.12.0.bb => qemu_3.0.0.bb} (88%)

diff --git a/meta/conf/distro/include/tcmode-default.inc b/meta/conf/distro/include/tcmode-default.inc
index bed1594b7a..1c9226cb06 100644
--- a/meta/conf/distro/include/tcmode-default.inc
+++ b/meta/conf/distro/include/tcmode-default.inc
@@ -28,7 +28,7 @@ BINUVERSION ?= "2.31%"
 GDBVERSION ?= "8.1%"
 GLIBCVERSION ?= "2.28%"
 LINUXLIBCVERSION ?= "4.15%"
-QEMUVERSION ?= "2.12%"
+QEMUVERSION ?= "3.0%"
 
 PREFERRED_VERSION_gcc ?= "${GCCVERSION}"
 PREFERRED_VERSION_gcc-cross-${TARGET_ARCH} ?= "${GCCVERSION}"
diff --git a/meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch b/meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch
deleted file mode 100644
index 837b65a79a..0000000000
--- a/meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch
+++ /dev/null
@@ -1,70 +0,0 @@
-CVE: CVE-2018-11806
-Upstream-Status: Backport
-
-https://lists.gnu.org/archive/html/qemu-devel/2018-06/msg01012.html
-
-From dc21a9d2951f0a2a7e63633e2b5c68c54e4edc4b Mon Sep 17 00:00:00 2001
-From: Jeremy Puhlman <jpuhlman@mvista.com>
-Date: Thu, 14 Jun 2018 01:28:49 +0000
-Subject: [PATCH] CVE-2018-11806 QEMU: slirp: heap buffer overflow
-
-Subject: [Qemu-devel] [PATCH 1/2] slirp: correct size computation while concatenating mbuf
-Date:	Tue, 5 Jun 2018 23:38:35 +0530
-From: Prasad J Pandit <address@hidden>
-
-While reassembling incoming fragmented datagrams, 'm_cat' routine
-extends the 'mbuf' buffer, if it has insufficient room. It computes
-a wrong buffer size, which leads to overwriting adjacent heap buffer
-area. Correct this size computation in m_cat.
-
-Reported-by: ZDI Disclosures <address@hidden>
-Signed-off-by: Prasad J Pandit <address@hidden>
----
- slirp/mbuf.c | 8 ++++----
- 1 file changed, 4 insertions(+), 4 deletions(-)
-
-Signed-off-by: Jeremy Puhlman <jpuhlman@mvista.com>
----
- slirp/mbuf.c | 8 ++++----
- 1 file changed, 4 insertions(+), 4 deletions(-)
-
-diff --git a/slirp/mbuf.c b/slirp/mbuf.c
-index 5ff2455..7fb4501 100644
---- a/slirp/mbuf.c
-+++ b/slirp/mbuf.c
-@@ -138,7 +138,7 @@ m_cat(struct mbuf *m, struct mbuf *n)
- 	 * If there's no room, realloc
- 	 */
- 	if (M_FREEROOM(m) < n->m_len)
--		m_inc(m,m->m_size+MINCSIZE);
-+               m_inc(m, m->m_len + n->m_len);
- 
- 	memcpy(m->m_data+m->m_len, n->m_data, n->m_len);
- 	m->m_len += n->m_len;
-@@ -158,12 +158,12 @@ m_inc(struct mbuf *m, int size)
- 
-         if (m->m_flags & M_EXT) {
- 	  datasize = m->m_data - m->m_ext;
--          m->m_ext = g_realloc(m->m_ext, size);
-+          m->m_ext = g_realloc(m->m_ext, size + datasize);
- 	  m->m_data = m->m_ext + datasize;
-         } else {
- 	  char *dat;
- 	  datasize = m->m_data - m->m_dat;
--          dat = g_malloc(size);
-+          dat = g_malloc(size + datasize);
- 	  memcpy(dat, m->m_dat, m->m_size);
- 
- 	  m->m_ext = dat;
-@@ -171,7 +171,7 @@ m_inc(struct mbuf *m, int size)
- 	  m->m_flags |= M_EXT;
-         }
- 
--        m->m_size = size;
-+        m->m_size = size + datasize;
- 
- }
- 
--- 
-2.13.3
-
diff --git a/meta/recipes-devtools/qemu/qemu_2.12.0.bb b/meta/recipes-devtools/qemu/qemu_3.0.0.bb
similarity index 88%
rename from meta/recipes-devtools/qemu/qemu_2.12.0.bb
rename to meta/recipes-devtools/qemu/qemu_3.0.0.bb
index 98db44a8ef..80da0810f1 100644
--- a/meta/recipes-devtools/qemu/qemu_2.12.0.bb
+++ b/meta/recipes-devtools/qemu/qemu_3.0.0.bb
@@ -7,7 +7,7 @@ RDEPENDS_${PN}-ptest = "bash make"
 LIC_FILES_CHKSUM = "file://COPYING;md5=441c28d2cf86e15a37fa47e15a72fbac \
                     file://COPYING.LIB;endline=24;md5=c04def7ae38850e7d3ef548588159913"
 
-SRC_URI = "http://wiki.qemu-project.org/download/${BP}.tar.bz2 \
+SRC_URI = "https://download.qemu.org/${BPN}-${PV}.tar.xz \
            file://powerpc_rom.bin \
            file://0001-sdl.c-allow-user-to-disable-pointer-grabs.patch \
            file://0002-qemu-Add-missing-wacom-HID-descriptor.patch \
@@ -21,7 +21,6 @@ SRC_URI = "http://wiki.qemu-project.org/download/${BP}.tar.bz2 \
            file://0009-apic-fixup-fallthrough-to-PIC.patch \
            file://0010-linux-user-Fix-webkitgtk-hangs-on-32-bit-x86-target.patch \
            file://0011-Revert-linux-user-fix-mmap-munmap-mprotect-mremap-sh.patch \
-           file://0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch \
            "
 UPSTREAM_CHECK_REGEX = "qemu-(?P<pver>\d+\..*)\.tar"
 
@@ -30,8 +29,8 @@ SRC_URI_append_class-native = " \
             file://0013-cpus.c-Add-error-messages-when-qemi_cpu_kick_thread-.patch \
             "
 
-SRC_URI[md5sum] = "122fd7cdf241ff7eb1e42c0b503b5d1b"
-SRC_URI[sha256sum] = "c9f4a147bc915d24df9784affc611a115f42d24720a89210b479f1ba7a3f679c"
+SRC_URI[md5sum] = "6a5c8df583406ea24ef25b239c3243e0"
+SRC_URI[sha256sum] = "8d7af64fe8bd5ea5c3bdf17131a8b858491bcce1ee3839425a6d91fb821b5713"
 
 COMPATIBLE_HOST_mipsarchn32 = "null"
 COMPATIBLE_HOST_mipsarchn64 = "null"
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] recipes-devtools: Upgrade QEMU to 3.0
  2018-08-15 22:25 [PATCH] recipes-devtools: Upgrade QEMU to 3.0 Alistair Francis
@ 2018-08-16  4:05 ` Randy MacLeod
  2018-08-16  5:58   ` Khem Raj
  0 siblings, 1 reply; 4+ messages in thread
From: Randy MacLeod @ 2018-08-16  4:05 UTC (permalink / raw)
  To: Alistair Francis, openembedded-core

LGTM.

Patched and built on an ubu-16.04 box.
I was able to log into both qemux86-64 and qemuppc
using core-image-minimal.

As people might expect, loads has changed:
qemu.git $ git diff v2.12.0..v3.0.0| diffstat| tail -1
  1825 files changed, 101360 insertions(+), 46333 deletions(-)

Have you tested all of the oe-core qemu archs with the upgrade?

Any concerns other than what is listed here:
    https://wiki.qemu.org/ChangeLog/3.0

../Randy


On 08/15/2018 06:25 PM, Alistair Francis wrote:
> Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
> ---
>   meta/conf/distro/include/tcmode-default.inc   |  2 +-
>   ...1806-QEMU-slirp-heap-buffer-overflow.patch | 70 -------------------
>   .../qemu/{qemu_2.12.0.bb => qemu_3.0.0.bb}    |  7 +-
>   3 files changed, 4 insertions(+), 75 deletions(-)
>   delete mode 100644 meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch
>   rename meta/recipes-devtools/qemu/{qemu_2.12.0.bb => qemu_3.0.0.bb} (88%)
> 
> diff --git a/meta/conf/distro/include/tcmode-default.inc b/meta/conf/distro/include/tcmode-default.inc
> index bed1594b7a..1c9226cb06 100644
> --- a/meta/conf/distro/include/tcmode-default.inc
> +++ b/meta/conf/distro/include/tcmode-default.inc
> @@ -28,7 +28,7 @@ BINUVERSION ?= "2.31%"
>   GDBVERSION ?= "8.1%"
>   GLIBCVERSION ?= "2.28%"
>   LINUXLIBCVERSION ?= "4.15%"
> -QEMUVERSION ?= "2.12%"
> +QEMUVERSION ?= "3.0%"
>   
>   PREFERRED_VERSION_gcc ?= "${GCCVERSION}"
>   PREFERRED_VERSION_gcc-cross-${TARGET_ARCH} ?= "${GCCVERSION}"
> diff --git a/meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch b/meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch
> deleted file mode 100644
> index 837b65a79a..0000000000
> --- a/meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch
> +++ /dev/null
> @@ -1,70 +0,0 @@
> -CVE: CVE-2018-11806
> -Upstream-Status: Backport
> -
> -https://lists.gnu.org/archive/html/qemu-devel/2018-06/msg01012.html
> -
> -From dc21a9d2951f0a2a7e63633e2b5c68c54e4edc4b Mon Sep 17 00:00:00 2001
> -From: Jeremy Puhlman <jpuhlman@mvista.com>
> -Date: Thu, 14 Jun 2018 01:28:49 +0000
> -Subject: [PATCH] CVE-2018-11806 QEMU: slirp: heap buffer overflow
> -
> -Subject: [Qemu-devel] [PATCH 1/2] slirp: correct size computation while concatenating mbuf
> -Date:	Tue, 5 Jun 2018 23:38:35 +0530
> -From: Prasad J Pandit <address@hidden>
> -
> -While reassembling incoming fragmented datagrams, 'm_cat' routine
> -extends the 'mbuf' buffer, if it has insufficient room. It computes
> -a wrong buffer size, which leads to overwriting adjacent heap buffer
> -area. Correct this size computation in m_cat.
> -
> -Reported-by: ZDI Disclosures <address@hidden>
> -Signed-off-by: Prasad J Pandit <address@hidden>
> ----
> - slirp/mbuf.c | 8 ++++----
> - 1 file changed, 4 insertions(+), 4 deletions(-)
> -
> -Signed-off-by: Jeremy Puhlman <jpuhlman@mvista.com>
> ----
> - slirp/mbuf.c | 8 ++++----
> - 1 file changed, 4 insertions(+), 4 deletions(-)
> -
> -diff --git a/slirp/mbuf.c b/slirp/mbuf.c
> -index 5ff2455..7fb4501 100644
> ---- a/slirp/mbuf.c
> -+++ b/slirp/mbuf.c
> -@@ -138,7 +138,7 @@ m_cat(struct mbuf *m, struct mbuf *n)
> - 	 * If there's no room, realloc
> - 	 */
> - 	if (M_FREEROOM(m) < n->m_len)
> --		m_inc(m,m->m_size+MINCSIZE);
> -+               m_inc(m, m->m_len + n->m_len);
> -
> - 	memcpy(m->m_data+m->m_len, n->m_data, n->m_len);
> - 	m->m_len += n->m_len;
> -@@ -158,12 +158,12 @@ m_inc(struct mbuf *m, int size)
> -
> -         if (m->m_flags & M_EXT) {
> - 	  datasize = m->m_data - m->m_ext;
> --          m->m_ext = g_realloc(m->m_ext, size);
> -+          m->m_ext = g_realloc(m->m_ext, size + datasize);
> - 	  m->m_data = m->m_ext + datasize;
> -         } else {
> - 	  char *dat;
> - 	  datasize = m->m_data - m->m_dat;
> --          dat = g_malloc(size);
> -+          dat = g_malloc(size + datasize);
> - 	  memcpy(dat, m->m_dat, m->m_size);
> -
> - 	  m->m_ext = dat;
> -@@ -171,7 +171,7 @@ m_inc(struct mbuf *m, int size)
> - 	  m->m_flags |= M_EXT;
> -         }
> -
> --        m->m_size = size;
> -+        m->m_size = size + datasize;
> -
> - }
> -
> ---
> -2.13.3
> -
> diff --git a/meta/recipes-devtools/qemu/qemu_2.12.0.bb b/meta/recipes-devtools/qemu/qemu_3.0.0.bb
> similarity index 88%
> rename from meta/recipes-devtools/qemu/qemu_2.12.0.bb
> rename to meta/recipes-devtools/qemu/qemu_3.0.0.bb
> index 98db44a8ef..80da0810f1 100644
> --- a/meta/recipes-devtools/qemu/qemu_2.12.0.bb
> +++ b/meta/recipes-devtools/qemu/qemu_3.0.0.bb
> @@ -7,7 +7,7 @@ RDEPENDS_${PN}-ptest = "bash make"
>   LIC_FILES_CHKSUM = "file://COPYING;md5=441c28d2cf86e15a37fa47e15a72fbac \
>                       file://COPYING.LIB;endline=24;md5=c04def7ae38850e7d3ef548588159913"
>   
> -SRC_URI = "http://wiki.qemu-project.org/download/${BP}.tar.bz2 \
> +SRC_URI = "https://download.qemu.org/${BPN}-${PV}.tar.xz \
>              file://powerpc_rom.bin \
>              file://0001-sdl.c-allow-user-to-disable-pointer-grabs.patch \
>              file://0002-qemu-Add-missing-wacom-HID-descriptor.patch \
> @@ -21,7 +21,6 @@ SRC_URI = "http://wiki.qemu-project.org/download/${BP}.tar.bz2 \
>              file://0009-apic-fixup-fallthrough-to-PIC.patch \
>              file://0010-linux-user-Fix-webkitgtk-hangs-on-32-bit-x86-target.patch \
>              file://0011-Revert-linux-user-fix-mmap-munmap-mprotect-mremap-sh.patch \
> -           file://0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch \
>              "
>   UPSTREAM_CHECK_REGEX = "qemu-(?P<pver>\d+\..*)\.tar"
>   
> @@ -30,8 +29,8 @@ SRC_URI_append_class-native = " \
>               file://0013-cpus.c-Add-error-messages-when-qemi_cpu_kick_thread-.patch \
>               "
>   
> -SRC_URI[md5sum] = "122fd7cdf241ff7eb1e42c0b503b5d1b"
> -SRC_URI[sha256sum] = "c9f4a147bc915d24df9784affc611a115f42d24720a89210b479f1ba7a3f679c"
> +SRC_URI[md5sum] = "6a5c8df583406ea24ef25b239c3243e0"
> +SRC_URI[sha256sum] = "8d7af64fe8bd5ea5c3bdf17131a8b858491bcce1ee3839425a6d91fb821b5713"
>   
>   COMPATIBLE_HOST_mipsarchn32 = "null"
>   COMPATIBLE_HOST_mipsarchn64 = "null"
> 


-- 
# Randy MacLeod
# Wind River Linux


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] recipes-devtools: Upgrade QEMU to 3.0
  2018-08-16  4:05 ` Randy MacLeod
@ 2018-08-16  5:58   ` Khem Raj
  2018-08-16 15:59     ` Alistair Francis
  0 siblings, 1 reply; 4+ messages in thread
From: Khem Raj @ 2018-08-16  5:58 UTC (permalink / raw)
  To: Randy MacLeod; +Cc: Patches and discussions about the oe-core layer

On Wed, Aug 15, 2018 at 9:05 PM Randy MacLeod
<randy.macleod@windriver.com> wrote:
>
> LGTM.
>
> Patched and built on an ubu-16.04 box.
> I was able to log into both qemux86-64 and qemuppc
> using core-image-minimal.
>
> As people might expect, loads has changed:
> qemu.git $ git diff v2.12.0..v3.0.0| diffstat| tail -1
>   1825 files changed, 101360 insertions(+), 46333 deletions(-)
>
> Have you tested all of the oe-core qemu archs with the upgrade?
>
> Any concerns other than what is listed here:
>     https://wiki.qemu.org/ChangeLog/3.0
>

incompatible changes and deprecated options seems ok since we do not
use any of them
other changes nothing is eye popping.

> ../Randy
>
>
> On 08/15/2018 06:25 PM, Alistair Francis wrote:
> > Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
> > ---
> >   meta/conf/distro/include/tcmode-default.inc   |  2 +-
> >   ...1806-QEMU-slirp-heap-buffer-overflow.patch | 70 -------------------
> >   .../qemu/{qemu_2.12.0.bb => qemu_3.0.0.bb}    |  7 +-
> >   3 files changed, 4 insertions(+), 75 deletions(-)
> >   delete mode 100644 meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch
> >   rename meta/recipes-devtools/qemu/{qemu_2.12.0.bb => qemu_3.0.0.bb} (88%)
> >
> > diff --git a/meta/conf/distro/include/tcmode-default.inc b/meta/conf/distro/include/tcmode-default.inc
> > index bed1594b7a..1c9226cb06 100644
> > --- a/meta/conf/distro/include/tcmode-default.inc
> > +++ b/meta/conf/distro/include/tcmode-default.inc
> > @@ -28,7 +28,7 @@ BINUVERSION ?= "2.31%"
> >   GDBVERSION ?= "8.1%"
> >   GLIBCVERSION ?= "2.28%"
> >   LINUXLIBCVERSION ?= "4.15%"
> > -QEMUVERSION ?= "2.12%"
> > +QEMUVERSION ?= "3.0%"
> >
> >   PREFERRED_VERSION_gcc ?= "${GCCVERSION}"
> >   PREFERRED_VERSION_gcc-cross-${TARGET_ARCH} ?= "${GCCVERSION}"
> > diff --git a/meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch b/meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch
> > deleted file mode 100644
> > index 837b65a79a..0000000000
> > --- a/meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch
> > +++ /dev/null
> > @@ -1,70 +0,0 @@
> > -CVE: CVE-2018-11806
> > -Upstream-Status: Backport
> > -
> > -https://lists.gnu.org/archive/html/qemu-devel/2018-06/msg01012.html
> > -
> > -From dc21a9d2951f0a2a7e63633e2b5c68c54e4edc4b Mon Sep 17 00:00:00 2001
> > -From: Jeremy Puhlman <jpuhlman@mvista.com>
> > -Date: Thu, 14 Jun 2018 01:28:49 +0000
> > -Subject: [PATCH] CVE-2018-11806 QEMU: slirp: heap buffer overflow
> > -
> > -Subject: [Qemu-devel] [PATCH 1/2] slirp: correct size computation while concatenating mbuf
> > -Date:        Tue, 5 Jun 2018 23:38:35 +0530
> > -From: Prasad J Pandit <address@hidden>
> > -
> > -While reassembling incoming fragmented datagrams, 'm_cat' routine
> > -extends the 'mbuf' buffer, if it has insufficient room. It computes
> > -a wrong buffer size, which leads to overwriting adjacent heap buffer
> > -area. Correct this size computation in m_cat.
> > -
> > -Reported-by: ZDI Disclosures <address@hidden>
> > -Signed-off-by: Prasad J Pandit <address@hidden>
> > ----
> > - slirp/mbuf.c | 8 ++++----
> > - 1 file changed, 4 insertions(+), 4 deletions(-)
> > -
> > -Signed-off-by: Jeremy Puhlman <jpuhlman@mvista.com>
> > ----
> > - slirp/mbuf.c | 8 ++++----
> > - 1 file changed, 4 insertions(+), 4 deletions(-)
> > -
> > -diff --git a/slirp/mbuf.c b/slirp/mbuf.c
> > -index 5ff2455..7fb4501 100644
> > ---- a/slirp/mbuf.c
> > -+++ b/slirp/mbuf.c
> > -@@ -138,7 +138,7 @@ m_cat(struct mbuf *m, struct mbuf *n)
> > -      * If there's no room, realloc
> > -      */
> > -     if (M_FREEROOM(m) < n->m_len)
> > --            m_inc(m,m->m_size+MINCSIZE);
> > -+               m_inc(m, m->m_len + n->m_len);
> > -
> > -     memcpy(m->m_data+m->m_len, n->m_data, n->m_len);
> > -     m->m_len += n->m_len;
> > -@@ -158,12 +158,12 @@ m_inc(struct mbuf *m, int size)
> > -
> > -         if (m->m_flags & M_EXT) {
> > -       datasize = m->m_data - m->m_ext;
> > --          m->m_ext = g_realloc(m->m_ext, size);
> > -+          m->m_ext = g_realloc(m->m_ext, size + datasize);
> > -       m->m_data = m->m_ext + datasize;
> > -         } else {
> > -       char *dat;
> > -       datasize = m->m_data - m->m_dat;
> > --          dat = g_malloc(size);
> > -+          dat = g_malloc(size + datasize);
> > -       memcpy(dat, m->m_dat, m->m_size);
> > -
> > -       m->m_ext = dat;
> > -@@ -171,7 +171,7 @@ m_inc(struct mbuf *m, int size)
> > -       m->m_flags |= M_EXT;
> > -         }
> > -
> > --        m->m_size = size;
> > -+        m->m_size = size + datasize;
> > -
> > - }
> > -
> > ---
> > -2.13.3
> > -
> > diff --git a/meta/recipes-devtools/qemu/qemu_2.12.0.bb b/meta/recipes-devtools/qemu/qemu_3.0.0.bb
> > similarity index 88%
> > rename from meta/recipes-devtools/qemu/qemu_2.12.0.bb
> > rename to meta/recipes-devtools/qemu/qemu_3.0.0.bb
> > index 98db44a8ef..80da0810f1 100644
> > --- a/meta/recipes-devtools/qemu/qemu_2.12.0.bb
> > +++ b/meta/recipes-devtools/qemu/qemu_3.0.0.bb
> > @@ -7,7 +7,7 @@ RDEPENDS_${PN}-ptest = "bash make"
> >   LIC_FILES_CHKSUM = "file://COPYING;md5=441c28d2cf86e15a37fa47e15a72fbac \
> >                       file://COPYING.LIB;endline=24;md5=c04def7ae38850e7d3ef548588159913"
> >
> > -SRC_URI = "http://wiki.qemu-project.org/download/${BP}.tar.bz2 \
> > +SRC_URI = "https://download.qemu.org/${BPN}-${PV}.tar.xz \
> >              file://powerpc_rom.bin \
> >              file://0001-sdl.c-allow-user-to-disable-pointer-grabs.patch \
> >              file://0002-qemu-Add-missing-wacom-HID-descriptor.patch \
> > @@ -21,7 +21,6 @@ SRC_URI = "http://wiki.qemu-project.org/download/${BP}.tar.bz2 \
> >              file://0009-apic-fixup-fallthrough-to-PIC.patch \
> >              file://0010-linux-user-Fix-webkitgtk-hangs-on-32-bit-x86-target.patch \
> >              file://0011-Revert-linux-user-fix-mmap-munmap-mprotect-mremap-sh.patch \
> > -           file://0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch \
> >              "
> >   UPSTREAM_CHECK_REGEX = "qemu-(?P<pver>\d+\..*)\.tar"
> >
> > @@ -30,8 +29,8 @@ SRC_URI_append_class-native = " \
> >               file://0013-cpus.c-Add-error-messages-when-qemi_cpu_kick_thread-.patch \
> >               "
> >
> > -SRC_URI[md5sum] = "122fd7cdf241ff7eb1e42c0b503b5d1b"
> > -SRC_URI[sha256sum] = "c9f4a147bc915d24df9784affc611a115f42d24720a89210b479f1ba7a3f679c"
> > +SRC_URI[md5sum] = "6a5c8df583406ea24ef25b239c3243e0"
> > +SRC_URI[sha256sum] = "8d7af64fe8bd5ea5c3bdf17131a8b858491bcce1ee3839425a6d91fb821b5713"
> >
> >   COMPATIBLE_HOST_mipsarchn32 = "null"
> >   COMPATIBLE_HOST_mipsarchn64 = "null"
> >
>
>
> --
> # Randy MacLeod
> # Wind River Linux
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] recipes-devtools: Upgrade QEMU to 3.0
  2018-08-16  5:58   ` Khem Raj
@ 2018-08-16 15:59     ` Alistair Francis
  0 siblings, 0 replies; 4+ messages in thread
From: Alistair Francis @ 2018-08-16 15:59 UTC (permalink / raw)
  To: Khem Raj; +Cc: Patches and discussions about the oe-core layer

On Wed, Aug 15, 2018 at 10:58 PM, Khem Raj <raj.khem@gmail.com> wrote:
> On Wed, Aug 15, 2018 at 9:05 PM Randy MacLeod
> <randy.macleod@windriver.com> wrote:
>>
>> LGTM.
>>
>> Patched and built on an ubu-16.04 box.
>> I was able to log into both qemux86-64 and qemuppc
>> using core-image-minimal.
>>
>> As people might expect, loads has changed:
>> qemu.git $ git diff v2.12.0..v3.0.0| diffstat| tail -1
>>   1825 files changed, 101360 insertions(+), 46333 deletions(-)
>>
>> Have you tested all of the oe-core qemu archs with the upgrade?

I tested it with x86 and RISC-V, no problems so far. I've been using
the RC for awhile now.

>>
>> Any concerns other than what is listed here:
>>     https://wiki.qemu.org/ChangeLog/3.0
>>
>
> incompatible changes and deprecated options seems ok since we do not
> use any of them
> other changes nothing is eye popping.

Agreed, I haven't seen anything that should cause alarm. Mostly just
improvements.

Alistair

>
>> ../Randy
>>
>>
>> On 08/15/2018 06:25 PM, Alistair Francis wrote:
>> > Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
>> > ---
>> >   meta/conf/distro/include/tcmode-default.inc   |  2 +-
>> >   ...1806-QEMU-slirp-heap-buffer-overflow.patch | 70 -------------------
>> >   .../qemu/{qemu_2.12.0.bb => qemu_3.0.0.bb}    |  7 +-
>> >   3 files changed, 4 insertions(+), 75 deletions(-)
>> >   delete mode 100644 meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch
>> >   rename meta/recipes-devtools/qemu/{qemu_2.12.0.bb => qemu_3.0.0.bb} (88%)
>> >
>> > diff --git a/meta/conf/distro/include/tcmode-default.inc b/meta/conf/distro/include/tcmode-default.inc
>> > index bed1594b7a..1c9226cb06 100644
>> > --- a/meta/conf/distro/include/tcmode-default.inc
>> > +++ b/meta/conf/distro/include/tcmode-default.inc
>> > @@ -28,7 +28,7 @@ BINUVERSION ?= "2.31%"
>> >   GDBVERSION ?= "8.1%"
>> >   GLIBCVERSION ?= "2.28%"
>> >   LINUXLIBCVERSION ?= "4.15%"
>> > -QEMUVERSION ?= "2.12%"
>> > +QEMUVERSION ?= "3.0%"
>> >
>> >   PREFERRED_VERSION_gcc ?= "${GCCVERSION}"
>> >   PREFERRED_VERSION_gcc-cross-${TARGET_ARCH} ?= "${GCCVERSION}"
>> > diff --git a/meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch b/meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch
>> > deleted file mode 100644
>> > index 837b65a79a..0000000000
>> > --- a/meta/recipes-devtools/qemu/qemu/0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch
>> > +++ /dev/null
>> > @@ -1,70 +0,0 @@
>> > -CVE: CVE-2018-11806
>> > -Upstream-Status: Backport
>> > -
>> > -https://lists.gnu.org/archive/html/qemu-devel/2018-06/msg01012.html
>> > -
>> > -From dc21a9d2951f0a2a7e63633e2b5c68c54e4edc4b Mon Sep 17 00:00:00 2001
>> > -From: Jeremy Puhlman <jpuhlman@mvista.com>
>> > -Date: Thu, 14 Jun 2018 01:28:49 +0000
>> > -Subject: [PATCH] CVE-2018-11806 QEMU: slirp: heap buffer overflow
>> > -
>> > -Subject: [Qemu-devel] [PATCH 1/2] slirp: correct size computation while concatenating mbuf
>> > -Date:        Tue, 5 Jun 2018 23:38:35 +0530
>> > -From: Prasad J Pandit <address@hidden>
>> > -
>> > -While reassembling incoming fragmented datagrams, 'm_cat' routine
>> > -extends the 'mbuf' buffer, if it has insufficient room. It computes
>> > -a wrong buffer size, which leads to overwriting adjacent heap buffer
>> > -area. Correct this size computation in m_cat.
>> > -
>> > -Reported-by: ZDI Disclosures <address@hidden>
>> > -Signed-off-by: Prasad J Pandit <address@hidden>
>> > ----
>> > - slirp/mbuf.c | 8 ++++----
>> > - 1 file changed, 4 insertions(+), 4 deletions(-)
>> > -
>> > -Signed-off-by: Jeremy Puhlman <jpuhlman@mvista.com>
>> > ----
>> > - slirp/mbuf.c | 8 ++++----
>> > - 1 file changed, 4 insertions(+), 4 deletions(-)
>> > -
>> > -diff --git a/slirp/mbuf.c b/slirp/mbuf.c
>> > -index 5ff2455..7fb4501 100644
>> > ---- a/slirp/mbuf.c
>> > -+++ b/slirp/mbuf.c
>> > -@@ -138,7 +138,7 @@ m_cat(struct mbuf *m, struct mbuf *n)
>> > -      * If there's no room, realloc
>> > -      */
>> > -     if (M_FREEROOM(m) < n->m_len)
>> > --            m_inc(m,m->m_size+MINCSIZE);
>> > -+               m_inc(m, m->m_len + n->m_len);
>> > -
>> > -     memcpy(m->m_data+m->m_len, n->m_data, n->m_len);
>> > -     m->m_len += n->m_len;
>> > -@@ -158,12 +158,12 @@ m_inc(struct mbuf *m, int size)
>> > -
>> > -         if (m->m_flags & M_EXT) {
>> > -       datasize = m->m_data - m->m_ext;
>> > --          m->m_ext = g_realloc(m->m_ext, size);
>> > -+          m->m_ext = g_realloc(m->m_ext, size + datasize);
>> > -       m->m_data = m->m_ext + datasize;
>> > -         } else {
>> > -       char *dat;
>> > -       datasize = m->m_data - m->m_dat;
>> > --          dat = g_malloc(size);
>> > -+          dat = g_malloc(size + datasize);
>> > -       memcpy(dat, m->m_dat, m->m_size);
>> > -
>> > -       m->m_ext = dat;
>> > -@@ -171,7 +171,7 @@ m_inc(struct mbuf *m, int size)
>> > -       m->m_flags |= M_EXT;
>> > -         }
>> > -
>> > --        m->m_size = size;
>> > -+        m->m_size = size + datasize;
>> > -
>> > - }
>> > -
>> > ---
>> > -2.13.3
>> > -
>> > diff --git a/meta/recipes-devtools/qemu/qemu_2.12.0.bb b/meta/recipes-devtools/qemu/qemu_3.0.0.bb
>> > similarity index 88%
>> > rename from meta/recipes-devtools/qemu/qemu_2.12.0.bb
>> > rename to meta/recipes-devtools/qemu/qemu_3.0.0.bb
>> > index 98db44a8ef..80da0810f1 100644
>> > --- a/meta/recipes-devtools/qemu/qemu_2.12.0.bb
>> > +++ b/meta/recipes-devtools/qemu/qemu_3.0.0.bb
>> > @@ -7,7 +7,7 @@ RDEPENDS_${PN}-ptest = "bash make"
>> >   LIC_FILES_CHKSUM = "file://COPYING;md5=441c28d2cf86e15a37fa47e15a72fbac \
>> >                       file://COPYING.LIB;endline=24;md5=c04def7ae38850e7d3ef548588159913"
>> >
>> > -SRC_URI = "http://wiki.qemu-project.org/download/${BP}.tar.bz2 \
>> > +SRC_URI = "https://download.qemu.org/${BPN}-${PV}.tar.xz \
>> >              file://powerpc_rom.bin \
>> >              file://0001-sdl.c-allow-user-to-disable-pointer-grabs.patch \
>> >              file://0002-qemu-Add-missing-wacom-HID-descriptor.patch \
>> > @@ -21,7 +21,6 @@ SRC_URI = "http://wiki.qemu-project.org/download/${BP}.tar.bz2 \
>> >              file://0009-apic-fixup-fallthrough-to-PIC.patch \
>> >              file://0010-linux-user-Fix-webkitgtk-hangs-on-32-bit-x86-target.patch \
>> >              file://0011-Revert-linux-user-fix-mmap-munmap-mprotect-mremap-sh.patch \
>> > -           file://0001-CVE-2018-11806-QEMU-slirp-heap-buffer-overflow.patch \
>> >              "
>> >   UPSTREAM_CHECK_REGEX = "qemu-(?P<pver>\d+\..*)\.tar"
>> >
>> > @@ -30,8 +29,8 @@ SRC_URI_append_class-native = " \
>> >               file://0013-cpus.c-Add-error-messages-when-qemi_cpu_kick_thread-.patch \
>> >               "
>> >
>> > -SRC_URI[md5sum] = "122fd7cdf241ff7eb1e42c0b503b5d1b"
>> > -SRC_URI[sha256sum] = "c9f4a147bc915d24df9784affc611a115f42d24720a89210b479f1ba7a3f679c"
>> > +SRC_URI[md5sum] = "6a5c8df583406ea24ef25b239c3243e0"
>> > +SRC_URI[sha256sum] = "8d7af64fe8bd5ea5c3bdf17131a8b858491bcce1ee3839425a6d91fb821b5713"
>> >
>> >   COMPATIBLE_HOST_mipsarchn32 = "null"
>> >   COMPATIBLE_HOST_mipsarchn64 = "null"
>> >
>>
>>
>> --
>> # Randy MacLeod
>> # Wind River Linux
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2018-08-16 15:59 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-08-15 22:25 [PATCH] recipes-devtools: Upgrade QEMU to 3.0 Alistair Francis
2018-08-16  4:05 ` Randy MacLeod
2018-08-16  5:58   ` Khem Raj
2018-08-16 15:59     ` Alistair Francis

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.