[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of
From: |
Jim Meyering |
Subject: |
Re: [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup |
Date: |
Wed, 30 May 2012 09:48:55 +0200 |
Anthony Liguori wrote:
> On 05/30/2012 03:12 PM, Jim Meyering wrote:
>> Anthony Liguori wrote:
>>> On 05/09/2012 10:12 PM, Jim Meyering wrote:
>>>> Peter Maydell wrote:
>>>>> On 9 May 2012 15:01, Jim Meyering<address@hidden> wrote:
>>>>>> From 402100deb7e27b1d7ac619ebac963f861fae91b0 Mon Sep 17 00:00:00 2001
>>>>>> From: Jim Meyering<address@hidden>
>>>>>> Date: Mon, 7 May 2012 18:34:26 +0200
>>>>>> Subject: [PATCH] linux-user: remove two unchecked uses of strdup
>>>>>>
>>>>>> Remove two uses of strdup (use g_path_get_basename instead),
>>>>>> and add a comment that this strncpy use is ok.
>>>>>>
>>>>>> Signed-off-by: Jim Meyering<address@hidden>
>>>>>
>>>>> This version
>>>>> Reviewed-by: Peter Maydell<address@hidden>
>>>>>
>>>>> but you'll need to retransmit it (presumably as part of a v2
>>>>> of this series) so it is in the proper format for a patch email
>>>>> (otherwise all the conversational chatter ends up in the git
>>>>> commit message).
>>>>
>>>> Thanks again.
>>>> I'm amending to reflect ACKs and Reviewed-by: comments as they come in,
>>>> as well as corrections like yours.
>>>>
>>>> I expect to repost the whole series (or any subset) when appropriate.
>>>> Or I can push it to a publicly-accessible repository somewhere.
>>>
>>> Has this gotten reposted? I don't see a new version on the list.
>>
>> Hi Anthony,
>> I've reposted (as V2, V3, etc.) any individual patch that required revision,
>> but have not reposted the series. I would have reposted, but I recall
>> someone saying that their tools managed quite well with the one-off
>> reposts.
>
> I corrected myself later as I misunderstood your question. It handles
> the acked-bys but not partial updates to a patch series.
>
>> Would you like me to repost just this one or the whole 22-patch series?
>
> Could you please repost the full series?
Sure. Rebased, confirmed it still compiles and passes "make check"
and reposted as v2.
FYI, this time I sent only to qemu-devel, rather than adding per-patch Cc's.
- [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup, (continued)
- [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup, Jim Meyering, 2012/05/09
- Re: [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup, Peter Maydell, 2012/05/09
- Re: [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup, Jim Meyering, 2012/05/09
- Re: [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup, Peter Maydell, 2012/05/09
- Re: [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup, Jim Meyering, 2012/05/09
- Re: [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup, Peter Maydell, 2012/05/09
- Re: [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup, Jim Meyering, 2012/05/09
- Re: [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup, Anthony Liguori, 2012/05/30
- Re: [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup, Jim Meyering, 2012/05/30
- Re: [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup, Anthony Liguori, 2012/05/30
- Re: [Qemu-devel] [PATCH 08/22] linux-user: remove two unchecked uses of strdup,
Jim Meyering <=
[Qemu-devel] [PATCH 03/22] vmdk: relative_path: avoid buffer overrun, Jim Meyering, 2012/05/09
[Qemu-devel] [PATCH 13/22] virtio-9p: avoid unwarranted use of strncpy, Jim Meyering, 2012/05/09
[Qemu-devel] [PATCH 05/22] lm32: avoid buffer overrun, Jim Meyering, 2012/05/09
[Qemu-devel] [PATCH 19/22] qcow2: mark this file's sole strncpy use as justified, Jim Meyering, 2012/05/09