Re: [eichin@kitten.gen.ma.us: Bug#22816: cfingerd mishandles .nofinger]


Martin Schulze (joey@kuolema.Infodrom.North.DE)
Sun, 31 May 1998 22:41:35 +0200


On Sun, May 31, 1998 at 10:05:53PM +0200, Martin Schulze wrote:

> > cfingerd is supposed to *display* the contents of ~user/.nofinger, not
> > just print nothing at all. With the stock cfingerd.conf, this doesn't
> > happen, as I think the following demonstrates sufficiently. This used
> > to work but I don't know how long ago. (I have some users that really
> > do wish to remain hidden [on another system], I'd *rather* have some
> > way to indicate "just display the contents of
> > /etc/cfingerd/nouser_banner.txt for this user" but that's a seperate
> > issue... it should instead at least work as documented....)
>
> This has been fixed, too. The wrong code was in standard.c

... in check_illegal()

Hmbf, das war nicht alles check_illegal() wird im gesamten Code
genau an drei Stellen aufgerufen. Dabei wird einmal true=TRUE
und zweimal true=FALSE erwartet. *Die* Logic zu verstehen dauert
etws. Da es zweimal falsch wahr, heute haben Minderheiten
vorrang. Nein, im Ernst die check-Routine gibt TRUE zurueck,
wenn alles geklappt hat, ergo ist true=TRUE korrekt.

Wenn wir den gesamten Code mal untersuchen sollten, dann muessen
wir auf solche Fehler auf jeden Fall achten und ggf die Verzweigungen
einmal hinterfragen.

Was mich ernsthaft wundert ist, dass diese Probleme mit check_illegal()
offenbar noch niemand vor mir bemerkt hat. Tsk.

Gruesse,

        Joey

-- 
  / Martin Schulze  *  joey@infodrom.north.de  *  26129 Oldenburg /
 /                                     http://home.pages.de/~joey/
/ Unix is user friendly ...  It's just picky about it's friends /




This archive was generated by hypermail 2.0b3 on Sun Aug 08 1999 - 10:46:55 CEST