From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1MGY4j-0005Yj-3D
	for qemu-devel@nongnu.org; Tue, 16 Jun 2009 08:47:42 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1MGY4d-0005XT-HO
	for qemu-devel@nongnu.org; Tue, 16 Jun 2009 08:47:39 -0400
Received: from [199.232.76.173] (port=39215 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1MGY4d-0005XD-5K
	for qemu-devel@nongnu.org; Tue, 16 Jun 2009 08:47:35 -0400
Received: from mx2.redhat.com ([66.187.237.31]:53400)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <lcapitulino@redhat.com>) id 1MGY4c-000403-M0
	for qemu-devel@nongnu.org; Tue, 16 Jun 2009 08:47:34 -0400
Date: Tue, 16 Jun 2009 09:47:24 -0300
From: Luiz Capitulino <lcapitulino@redhat.com>
Subject: Re: [Qemu-devel] Re: monitor: check for readline in monitor_event()
Message-ID: <20090616094724.2724f916@doriath>
In-Reply-To: <4A36CE52.6020100@web.de>
References: <20090615173858.31514f4f@doriath>
	<4A36CE52.6020100@web.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Jan Kiszka <jan.kiszka@web.de>
Cc: qemu-devel <qemu-devel@nongnu.org>

On Tue, 16 Jun 2009 00:42:26 +0200
Jan Kiszka <jan.kiszka@web.de> wrote:

> Luiz Capitulino wrote:
> > The call of readline_show_prompt() in CHR_EVENT_RESET's body will
> > trig a segfault if readline is not being used, because 'mon->rs'
> > will be NULL.
> > 
> > This fixes the problem by adding the proper check.
> > 
> > I've trigged this while playing with an off-tree code that disables
> > readline support, I'm not sure whether in-tree code can trig this.
> > 
> > Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
> > ---
> >  monitor.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/monitor.c b/monitor.c
> > index 6b45f6c..787101d 100644
> > --- a/monitor.c
> > +++ b/monitor.c
> > @@ -3002,7 +3002,7 @@ static void monitor_event(void *opaque, int
> > event) case CHR_EVENT_RESET:
> >          monitor_printf(mon, "QEMU %s monitor - type 'help' for
> > more " "information\n", QEMU_VERSION);
> > -        if (mon->chr->focus == 0)
> > +        if (mon->rs && mon->chr->focus == 0)
> >              readline_show_prompt(mon->rs);
> >          break;
> >      }
> 
> In-tree code is not affected as no monitor user will call
> qemu_chr_reset for the associated char device if there is no readline
> active as well. Yeah, secret de-facto rule.

 Ok.

> The patch is not incorrect, but I would like to understand the
> out-of-tree use case behind it. So you do want the info line printed,
> but provide your own readline processor? What kind of terminal is
> this?

 I'm working on a machine-friendly monitor for QEMU, which disables
readline.

 I didn't want to get into the details now, it will be better to
discuss the prototype I'm writing, should be finished soon.