#35 pptpd hangup with "Long config file line ignored"

open
nobody
Bug (28)
8
2014-12-18
2005-03-14
jose lopez
No

In case /etc/pptpd.conf does not finish with a newline,
pptpd hangs on startup eating 100% CPU. Tested in
Ubuntu Hoary (pptpd 1.2.1). Adding the missing newline
makes pptpd start fine.

From what I have seen, the bug might be in
configfile.c, function read_config_file.

The do loop logic needs to include the possibility of
files not ending in a newline. In the while just before
the "Long config file..." message, something like

    if (buffer[(len = strlen(buffer)) - 1] != '\n' &&

!feof(in)) {

where the && !feof(in) should be added.

The place I where I have located the bug is
http://66.102.9.104/search?q=cache:L8jDx_hZLg0J:cvs.sourceforge.net/viewcvs.py/poptop/poptop-1.1.4/configfile.c%3Frev%3D1.2++%22Long+config+file+line+ignored.%22+pptpd&hl=es
which is the cache of what seems to be a repository for
1.1.4 (NOT 1.2.1). I am not able to grab the source
package right now and the sf.net Web to CVS gateway
seems down at the moment.

Cheers, andvaranaut

Discussion

  • jose lopez

    jose lopez - 2005-03-14

    Logged In: YES
    user_id=388343

    I'm taking the liberty to assign this a high priority
    because it is a trivial fix, but the error is really
    annoying and puzzling...

     
  • Dr. Gianluigi Zanettini

    I found this page Googling for this exact problem. I confirm the issue.

     
  • James Cameron

    James Cameron - 2013-02-07

    Fixed in CVS.

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks