公司的機器最近遇到一個bug(其實這個bug一年前就出現過,只是未引起重視),現象是這樣的:在生產環境中,用戶連續打印票的時候,中間某張票的一個時間可能會出問題,該時間本來是一個未來的時間,卻被打印成了系統當前的時間.其他同事認爲是傳入的參數出錯了,加了大量的調試信息和日誌進去,結果去令人掉眼鏡,傳入的參數是完全正確的,但是結果去不是預期的.
最終排查BUG的任務轉到了我手裏,我簡單分析了下上層流程,並無問題,於是把問題定位在底層庫的FormatTime函數上,於是要了份FormatTime的代碼查看(吐槽下,封閉的代碼庫往往造就一些隱蔽的問題),FormatTime的實現很簡單,調用localtime函數,然後格式化輸出,查看localtime函數的原型,如下
struct tm *localtime(const time_t *timep);
問題就出在localtime這個函數上,從函數原型來看,這個函數返回了一個struct tm的指針,但是傳入參數並未有傳入tm參數,那麼就有3種情況
1,tm是內部malloc出來的
2,tm是一個全局變量
3,tm是一個局部static
第一種情可以首先排除,因爲調用完localtime後沒要求free tm,或者提供相應的free函數,第2 3種情況可以視爲一種情況,那即是:使用了內部全局變量!
那麼問題就來了,這個函數是個非線程安全的函數,在多線程的環境下使用,會發生重入的情況,即如果兩個線程同時調用localtime函數,函數的返回結果都會是最後一次調用localtime的結果.這是一個典型的函數重入BUG.解決方法:使用線程安全的localtime_r函數替換localtime.至此,問題解決
c語言中有不少函數都是非線程安全的,例如strtok,gethostbyname,看到返回值爲指針類型的函數時,都要留個心眼.