Menu

#8 Too many open files

open
nobody
None
5
2002-05-20
2002-05-20
Jesse Glick
No

If you use transconnect on a single app for a while,
you will eventually run out of open files and the app
may crash. Happened to me with Mozilla and proxying
NNTP; would work for quite a while, then Mozilla would
suddenly die.

Found out that tconn.c does this:

fp = fopen (configfile, "r")

but never closes it again. Suggest adding after the
while (fgets....) block and before the for (lnnum =
0...) block:

if (fclose (fp))
{
perror ("Could not close config file");
}

Doesn't seem to do any harm, time will tell if it
solves the crashes.

Discussion

  • Dwivedi Ajay kumar

    Logged In: YES
    user_id=71886

    Hi,
    What version of transconnect are you using? I think I patched
    this problem long ago. Try using 1.2 or 1.3 Beta. If the
    problem persists, please inform me.

    Cheers,
    Ajay

     
  • Jesse Glick

    Jesse Glick - 2002-05-22

    Logged In: YES
    user_id=257383

    I built it from sources in the CVS trunk, which are up to
    date: tconn.c rev 1.6. My sources do not include any call to
    fclose() unless I apply the above patch.

     
  • Dwivedi Ajay kumar

    Logged In: YES
    user_id=71886

    Sorry Buddy,
    I haven't updated the CVS for a long time (1 year) since I
    moved from the university. Because my proxy here did not
    allow me to use my own tool :(
    Try downloading from the summary page.
    I will update the CVS in my next release. Promise :)

    Ajay

     
  • Jesse Glick

    Jesse Glick - 2002-05-23

    Logged In: YES
    user_id=257383

    OK, I will try downloading the prepackaged release. But it
    would be good to at least put a warning into CVS that it is
    out of date! It makes it harder for people to send you
    patches and the like when the sources are obsolete.

     

Log in to post a comment.

MongoDB Logo MongoDB