Bug#736479: insserv code missing brackets for a conditional statement

[Finer, Howard]

Package: insserv
Version: 1.14.0-2

While troubleshooting an issue we were having with  installing some init.d 
scripts, we downloaded the source code to insserv and were looking at 
differences between the SuSU and Debian code.

the SuSE section reads:
                                    if (serv && (serv->attr.flags & 
SERV_ALREADY)) {
                                                xremove(dfd, d->d_name);
                                                if (--serv->attr.ref <= 0)
                                                    serv->attr.flags &= 
~SERV_ENABLED;
                                    }

And the Debian section reads:
                                    if (serv && (serv->attr.flags & 
SERV_ALREADY))
                                                xremove(dfd, d->d_name);
                                                if (--serv->attr.ref <= 0)
                                                    serv->attr.flags &= 
~SERV_ENABLED;

It seems that the highlighted lines should be part of the condition and that 
the braces were mistakenly deleted.

I am using Debian 6.0.7 (/etc/debian_version) with kernel version 2.6.32-48.


Thanks,
Howard Finer
Sonus Inc.