helpGNU Serveez - Support: sr #107607, SIGINT causes segfault when...


sr #107607: SIGINT causes segfault when configured with "thread safety"

Submitted by:  Thien-Thi Nguyen <ttn>
Submitted on:  Mon 28 Feb 2011 10:52:36 AM UTC  
Category:  None Priority:  5 - Normal
Severity:  5 - Blocker Status:  Done
Privacy:  Public Assigned to:  Thien-Thi Nguyen <ttn>
Open/Closed:  Open Operating System:  None

Add a New Comment(Rich Markup)

You are not logged in

Please log in, so followups can be emailed to you.


Sun 06 Mar 2011 08:45:19 PM UTC, comment #2:

Oops, typo in analysis: s/--enable-debug/--enable-thread-safety/

Thien-Thi Nguyen <ttn>
Project AdministratorIn charge of this item.
Sun 06 Mar 2011 08:44:01 PM UTC, comment #1:

Analysis: top level calls ‘svz_halt’, which "destroys" the mutex used around the stdio calls in ‘svz_log’, but (when ‘--enable-debug’) top level calls ‘svz_log’ again, which given the (default) mutex type, causes a failure to grab the mutex leading to another ‘svz_log’ call, leading to unbounded stack growth until crash.

The solution adopted was to add a valid bit for the mutex and only do lock/unlock operations if valid. The funcs ‘svz_boot’ and ‘svz_halt’ now set and clear this bit, respectively.

  • commit: 74776742fbe27bf79144d7667afeab6d206f64a8
  • title: "Fix bug: Make ‘svz_log’ work also after its mutex is destroyed"

This bug can be closed now.

Thien-Thi Nguyen <ttn>
Project AdministratorIn charge of this item.
Mon 28 Feb 2011 10:52:36 AM UTC, original submission:

This is with the 0.1.7 release. To reproduce:

Thien-Thi Nguyen <ttn>
Project AdministratorIn charge of this item.


(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:

No files currently attached


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -email is unavailable- added by ttn (Submitted the item)

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.




    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2011-03-06 ttn StatusConfirmed => Done

    Back to the top

    Powered by Savane 3.3