Skip to content
Snippets Groups Projects
  • Jason Parker's avatar
    28fbbde3
    Merged revisions 82344 via svnmerge from · 28fbbde3
    Jason Parker authored
    https://origsvn.digium.com/svn/asterisk/branches/1.4
    
    ........
    r82344 | qwell | 2007-09-13 15:11:40 -0500 (Thu, 13 Sep 2007) | 9 lines
    
    Fix a crash that could occur in cdr_csv when mutliple threads tried to close the same file.
    
    Do we actually need the locking here?  What happens if you open the same file twice, and
     two threads try to write to it at the same time?  Is fputs() going to write out the entire
     line at once?  I suspect that it could be possible for the second fopen to run during the
     first fputs, so the position could be in the middle of the previously written line...
    
    Issue 10347, initial patch by explidous (but I removed all of the paranoia stuff..)
    
    ........
    
    
    git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@82345 65c4cc65-6c06-0410-ace0-fbb531ad65f3
    28fbbde3
    History
    Merged revisions 82344 via svnmerge from
    Jason Parker authored
    https://origsvn.digium.com/svn/asterisk/branches/1.4
    
    ........
    r82344 | qwell | 2007-09-13 15:11:40 -0500 (Thu, 13 Sep 2007) | 9 lines
    
    Fix a crash that could occur in cdr_csv when mutliple threads tried to close the same file.
    
    Do we actually need the locking here?  What happens if you open the same file twice, and
     two threads try to write to it at the same time?  Is fputs() going to write out the entire
     line at once?  I suspect that it could be possible for the second fopen to run during the
     first fputs, so the position could be in the middle of the previously written line...
    
    Issue 10347, initial patch by explidous (but I removed all of the paranoia stuff..)
    
    ........
    
    
    git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@82345 65c4cc65-6c06-0410-ace0-fbb531ad65f3