This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [patch] find_and_open_script: Fix failed fdopen

From: Pedro Alves <palves at redhat dot com>
To: Jan Kratochvil <jan dot kratochvil at redhat dot com>
Cc: gdb-patches at sourceware dot org
Date: Mon, 23 Jan 2012 16:31:02 +0000
Subject: Re: [patch] find_and_open_script: Fix failed fdopen
References: <20120123160309.GA25174@host2.jankratochvil.net> <4F1D862D.1060608@redhat.com> <20120123161751.GA25494@host2.jankratochvil.net>

On 01/23/2012 04:17 PM, Jan Kratochvil wrote:
> On Mon, 23 Jan 2012 17:09:17 +0100, Pedro Alves wrote:
>> This do_cleanups doesn't look right.  From the patch context, there's
>> already do_cleanups(old_cleanups) call a bit above.
> Oops, thanks.  Also missing close there.

Thanks.  Looks correct to me now.

-- 
Pedro Alves

Follow-Ups:
- [commit] [patch] find_and_open_script: Fix failed fdopen
  - From: Jan Kratochvil

References:
- [patch] find_and_open_script: Fix failed fdopen
  - From: Jan Kratochvil
- Re: [patch] find_and_open_script: Fix failed fdopen
  - From: Pedro Alves
- Re: [patch] find_and_open_script: Fix failed fdopen
  - From: Jan Kratochvil

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]