bugFreeciv - Bugs: bug #20721, Recursive logging

 
 
Show feedback again

bug #20721: Recursive logging

Submitted by:  Marko Lindqvist <cazfi>
Submitted on:  Sun Apr 7 06:36:51 2013  
 
Category: generalSeverity: 3 - Normal
Priority: 5 - NormalStatus: Fixed
Assigned to: Marko Lindqvist <cazfi>Open/Closed: Closed
Release: Operating System: None
Planned Release: 2.4.0, 2.5.0Contains string changes: None

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

Tue Apr 30 19:01:35 2013, SVN revision 22796:

Do not call freelog recursively from convert_string()

See bug #20721

(Browse SVN revision 22796)

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Tue Apr 30 19:01:30 2013, SVN revision 22795:

Do not call freelog recursively from convert_string()

See bug #20721

(Browse SVN revision 22795)

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Sun Apr 28 09:24:23 2013, comment #1:

Attached patch replaces logging functions in convert_string() with fprintf().
I think S2_3 is safe from this problem as the logging mutex is not used there.

(file #17840)

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Sun Apr 7 06:36:51 2013, original submission:

Not sure what's the bug(s) here, but the whole does not work.

Noticed S2_4 autogame that hadn't proceeded single turn in a day. Attaching debugger revealed that it was trying to allocate logging mutex in log_real(). Without threaded ai type in use, deadlock between two threads seemed unlikely (that's the only mutex ever used!) And indeed backtrace shows that logging gets called recursively, so it was the upper level call that had already locked the mutex.

Latter logging is about invalid conversion in convert_string()

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #17840:  RecursiveLog.patch added by cazfi (1kB - text/x-diff)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by cazfi (Submitted the item)
  •  

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

    Only logged-in users can vote.

     

    Error: not logged in

     

     

    Follow 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue Apr 30 19:01:47 2013cazfiStatusReady For Test=>Fixed
      Assigned toNone=>cazfi
      Open/ClosedOpen=>Closed
    Sun Apr 28 09:24:23 2013cazfiAttached File-=>Added RecursiveLog.patch, #17840
      CategoryNone=>general
      StatusNone=>Ready For Test
      Planned Release=>2.4.0, 2.5.0
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup