[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/4] qemu-io: initialize progname with error_
From: |
Hitoshi Mitake |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/4] qemu-io: initialize progname with error_set_progname() |
Date: |
Mon, 02 Feb 2015 23:04:52 +0900 |
User-agent: |
Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (Goj$(D+W(B) APEL/10.8 Emacs/24.3 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO) |
At Mon, 2 Feb 2015 12:36:00 +0100,
Kevin Wolf wrote:
>
> Am 02.02.2015 um 10:51 hat Hitoshi Mitake geschrieben:
> > At Thu, 22 Jan 2015 18:08:11 +0900,
> > Hitoshi Mitake wrote:
> > >
> > > Calling error_get_progname() in the context of qemu-io can cause
> > > segmentation fault because qemu-io doesn't initialize its progname
> > > with error_set_progname(). This patch adds the initialization.
> > >
> > > Currently, the missing call of error_set_progname() doesn't cause any
> > > problems because qemu-io doesn't use error_get_progname(). This patch
> > > is a proactive action.
> > >
> > > Cc: Kevin Wolf <address@hidden>
> > > Cc: Stefan Hajnoczi <address@hidden>
> > > Cc: Markus Armbruster <address@hidden>
> > > Signed-off-by: Hitoshi Mitake <address@hidden>
> > > ---
> > > qemu-io.c | 1 +
> > > 1 file changed, 1 insertion(+)
> >
> > Hi Kevin, Stefan, could you pick this patch? sheepdog driver has a
> > pending patch [1] which depends on it. I believe at least qemu-io
> > should call error_set_progname() because block drivers can use the
> > qemu-error infrastructure.
>
> So are you planning to get the individual patches merged on their own
> instead of the whole series as one?
I thought this patchset can be proactive fix. So I'm separating the
patch for sheepdog driver from this one.
>
> > [1]
> > https://github.com/sheepdog/qemu/commit/a95c35e606a2a189e7dbaf645277c5c306b01e4b
>
> That patch looks wrong. Nobody guarantees that qemu-io is really called
> qemu-io. The user can name their binaries as they want and create
> symlinks with any name, and indeed names for qemu-img used by some
> distributions include 'qemu-img-kvm' and 'kvm-img'. You need to find a
> different way than checking binary file names.
I couldn't consider about the case, thanks for pointing! I'll seek
other method for distinguishing binary files.
Thanks,
Hitoshi
>
> Kevin