Help save net neutrality! Learn more.
Close

#23 Off-by-one in conf file name generation

v3.2.3
closed
None
5
2008-02-02
2008-01-24
Rob
No

When the wrapper is run with no configuration file specified (e.g., no arguments at all), it defaults to wrapper.conf (or whatever the wrapper executable is called).

But the translation from "wrapper" to "wrapper.conf" allocates one too few bytes (it's adding 5 characters plus a newline).

This resulted in some bizarre behavior on Linux, where paths of a certain length would get random characters appended.

The attached patch fixes this problem in wrapper.c.

_Rob

Discussion

  • Rob

    Rob - 2008-01-24

    wrapper.c patch

     
  • Leif Mortenson

    Leif Mortenson - 2008-02-02

    Logged In: YES
    user_id=228081
    Originator: NO

    Rob,
    Thanks for taking the time to track this down and send in a patch. It has been committed and will be in the 3.3.0 release.

    Cheers,
    Leif

     
  • Leif Mortenson

    Leif Mortenson - 2008-02-02
    • assigned_to: nobody --> mortenson
    • status: open --> closed
     

Log in to post a comment.