wavemon crashes with "Segmentation fault" if config-File cannot be written

Bug #209722 reported by PinKing
4
Affects Status Importance Assigned to Milestone
wavemon (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: wavemon

Hello,

i start wavemon 0.4.0b-11 on a system, were the folder is read-only, were wavemon will save his
config-file. So wavemon crashes with "Segmentation fault", if you try to save the file.

I think this in file conf.c function write_cf() will not clearly work:
...
 if ((fd = fopen(cfname, "w")) < 0) fatal_error("cannot open %s", cfname);
 ll_reset(cfld);
 while ((lp = ll_getall(cfld))) fputs(lp, fd);
 fclose(fd);
....

I think it is the same about the other fopen() functions in wavemon.

Hope you understand me. Excuse my bad english.

Yours sincerely, Pinking.

Tags: bitesize

Related branches

Daniel T Chen (crimsun)
Changed in wavemon:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Arnaud Soyez (weboide) wrote :

Here's the debdiff that fixes this bug and also fixes:
https://bugs.launchpad.net/ubuntu/+source/wavemon/+bug/209727

http://launchpadlibrarian.net/20786912/wavemon_0.4.0b-11ubuntu2.debdiff
This is a debdiff against 0.4.0b-11.

Built and tested.

Revision history for this message
Arnaud Soyez (weboide) wrote :
Changed in wavemon:
status: Confirmed → In Progress
assignee: nobody → weboide
Revision history for this message
Arnaud Soyez (weboide) wrote :
Changed in wavemon:
assignee: weboide → nobody
status: In Progress → Confirmed
Revision history for this message
Steve Stalcup (vorian) wrote :

uploaded.

Thanks for your contribution to Ubuntu!

Changed in wavemon:
status: Confirmed → Fix Committed
Revision history for this message
Arnaud Soyez (weboide) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package wavemon - 0.4.0b-11ubuntu1

---------------
wavemon (0.4.0b-11ubuntu1) jaunty; urgency=low

  * Added 12_write_conf_read_only.diff: fixes segmentation fault when writing
    into a read-only conf file. (LP: #209722)
  * Added 13_enc_key_displayed_fix.diff: fixes encryption key display bug (" 6"
    into "06" for example). (LP: #209727)
  * Debian/control:
    - bumped standards version to 3.8.0
    - added homepage field
    - added ${misc:Depends}
    - updated maintainer to MOTU

 -- Arnaud Soyez <email address hidden> Sun, 28 Dec 2008 23:17:48 -0500

Changed in wavemon:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.