On Jul 29, 12:33pm, j+nbsd@2005.salmi.ch (Jukka Salmi) wrote:
-- Subject: Re: /etc/login.conf required to exist after user(8) changes

| It's the valid_class() function which causes the problem because it
| calls err(3) if login.conf does not exist.

I will get rid of the exit.

| Another possible problem: the pointer used in the return expression
| is already free()ed.

It is used, but not de-referenced so it is not a problem. I.e. the value
at the end of the pointer is freed, but we are only checking if there
was a value.

| The attached patch fixes both problems.

Thanks,

christos