Tollef Fog Heen wrote:
> It's not like those are the only options available. As an example, like
> you said, make the FILE* a local variable in get_next and get_pw_nam.
> For those of us who don't want the caching, let us set the hash table
> size to 0, and all is good, for those who want it, let them set it to a
> bigger value, they get caching, but have to reload the module when the
> file is updated. Everybody is then happy.
>
> It seems like there is still a race condition even with your fix, since
> two threads might be executing the
>
> if (ht->fp) {
>
> statement at the same time, leading both to close the file. So your fix
> has narrowed the race, but not eliminated it.
Sure. Send a patch.
Alan DeKok.
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
