This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [Windows]Fix a bug which cause GDB.exe assert when try to run the inferior
- From: Pedro Alves <palves at redhat dot com>
- To: Yao Qi <yao at codesourcery dot com>
- Cc: asmwarrior <asmwarrior at gmail dot com>, gdb-patches at sourceware dot org
- Date: Fri, 21 Feb 2014 10:13:47 +0000
- Subject: Re: [Windows]Fix a bug which cause GDB.exe assert when try to run the inferior
- Authentication-results: sourceware.org; auth=none
- References: <530612A8 dot 4000903 at gmail dot com> <5306A9BC dot 4080701 at codesourcery dot com> <5306EE2A dot 2060103 at gmail dot com> <5306FA43 dot 1070408 at codesourcery dot com>
On 02/21/2014 07:03 AM, Yao Qi wrote:
> On 02/21/2014 02:11 PM, asmwarrior wrote:
>> Windows: Fix target_xfer_partial: Assertion `*xfered_len > 0' failed.
>> The failure was introduced in commit 9b409511d07fe375284701af34909fb539029caf, 2014-02-11.
>> gdb/windows-nat.c (windows_xfer_shared_libraries) : If len is 0, it should return TARGET_XFER_EOF.
>
> The format of the changelog entry isn't correct. On the other hand,
> changelog entry is about "what are changed", so "it should return
> TARGET_XFER_EOF" doesn't describe "what are changed" clearly. I suggest
> something below. Otherwise, I don't have comments.
Thanks You. This is definitely OK. Could you put it in?
- return TARGET_XFER_OK;
+ return len ? TARGET_XFER_OK : TARGET_XFER_EOF;
(Please write 'len != 0' though.)
>
> 2014-02-21 Yuanhui Zhang <asmwarrior@gmail.com>
>
> * windows-nat.c (windows_xfer_shared_libraries): Return
> TARGET_XFER_EOF if LEN is zero.
>
--
Pedro Alves