[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] sheepdog: fix confused return values

From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH] sheepdog: fix confused return values
Date: Wed, 25 Feb 2015 10:35:17 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)

Liu Yuan <address@hidden> writes:

> On Wed, Feb 18, 2015 at 09:35:07AM +0100, Markus Armbruster wrote:
>> Liu Yuan <address@hidden> writes:
>> > From: Liu Yuan <address@hidden>
>> >
>> > These functions mix up -1 and -errno in return values and would might cause
>> > trouble error handling in the call chain.
>> >
>> > This patch let them return -errno and add some comments.
>> >
>> > Reported-by: Markus Armbruster <address@hidden>
>> > Signed-off-by: Liu Yuan <address@hidden>
>> Did you review all functions returning negative errno, or just the two
>> that caught my eye?
> Umm, mostly the two you mentioned.

I encourage you to review the whole file for this error pattern!  In my
experience, bugs occur in clusters.  When you find one, there's a high
chance similar ones exist, and looking for them is good practice.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]