netdev
[Top] [All Lists]
<prev [Date] next>
[Advanced]
 <prev [Thread] next>

Re: [PATCH] rtnl_unlock/lock in sch_api.c

from [Catalin(ux aka Dino) BOIE] [Permanent Link][Original]
To: Thomas Graf <tgraf@xxxxxxx>
Subject: Re: [PATCH] rtnl_unlock/lock in sch_api.c
From: "Catalin(ux aka Dino) BOIE" <util@xxxxxxxxxxxxxxx>
Date: Tue, 29 Mar 2005 01:27:07 +0300 (EEST)
Cc: netdev@xxxxxxxxxxx, davem@xxxxxxxxxxxxx
In-reply-to: <20050328144701.GA3086@postel.suug.ch>
References: <Pine.LNX.4.62.0503281720430.20453@webhosting.rdsbv.ro> <20050328144701.GA3086@postel.suug.ch>
Sender: netdev-bounce@xxxxxxxxxxx 
On Mon, 28 Mar 2005, Thomas Graf wrote:

Looks good, except...
+ err = -EAGAIN; 

You're missing a goto err_out here, the ops == NULL won't catch it.

+                       }
                }
        }
 #endif

-       err = -EINVAL;
        if (ops == NULL)
                goto err_out;

With the above goto inserted there is no need to move the err = -EINVAL. 

Thanks, Thomas.

You are right.

I'm redoind the patch right now.

---
Catalin(ux aka Dino) BOIE
catab at deuroconsult.ro
http://kernel.umbrella.ro/

[More with this subject...]
<Prev in Thread] Current Thread [Next in Thread>
Previous by Date:  Re: [PATCH] rtnl_unlock/lock in sch_api.c, Catalin(ux aka Dino) BOIE
Next by Date:  Re: [Ksummit-2005-discuss] Summary of 2005 Kernel Summit Proposed Topics, Benjamin LaHaise
Previous by Thread:  Re: [PATCH] rtnl_unlock/lock in sch_api.c, Thomas Graf
Next by Thread:  Re: [PATCH] rtnl_unlock/lock in sch_api.c, Patrick McHardy
Indexes:  [Date] [Thread] [Top] [All Lists]