Bug in bb 1.5.0 ifupdown applet

[Natanael Copa]

On Wed, 2 May 2007 23:22:35 +0200
Denis Vlasenko <vda.linux at googlemail.com> wrote:

> Small question: after else below, we build new list,
> write it out if !NO_ACT, and then delete.
> 
> If NO_ACT is true, then list is build and then deleted
> without any useful effect.
> 
> Should we replace "else {...}" with "else if (!NO_ACT) {...}" ?

duh.. ofcourse... Yes I think we should.
Updated patch attatched.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: reentrant-ifupdown2.patch
Type: application/octet-stream
Size: 3363 bytes
Desc: not available
Url : http://busybox.net/lists/busybox/attachments/20070502/cbab58e2/attachment.obj