group.c bug

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

 




If any of you use group security (filtering out groups so specific people
cannot see them) then this applies to you.

group.c when denying a user access uses:
		emitrn (NNTP_ACCESS);
in two places.

This has the effect of returning a '502 Permission denied' *fatal* error.

More correctly this should be:
		emitrn (NNTP_NOSUCHGROUP);
which will return a "411 No such group." error.

tin users will find this fix particularly useful it quits on startup just
because a banned group is in its .newsrc.  This may also apply to other
'correctly' written newsreaders.

The fix is easily made as group.c only has two refs to NNTP_ACCESS.

Paul 'ihave winge-er' Gregg.

PS: thanks to Hugo.Embrechts@kulnet.kuleuven.ac.be for his ihave patch
which works well.

--
 The Internet Business Ltd, Holywood House, Innis Court, Holywood, BT18 9HF
 pgregg@tibus.net                                     http://www.tibus.net/
            Phone: +44 (0)1232-424190  Fax: +44 (0)1232-424709 
           Eight out of every five people are math illiterates.


[Index of Archives]     [Yosemite]     [Yosemite Campsites]     [Bugtraq]     [Linux]     [Trn]

Powered by Linux