來分析一個C語言代碼結構的設計問題。

這段代碼，使用了兩次malloc，分別給 p1 和 p2 申請了內存。用完后，內存釋放，防止內存泄漏。

大家覺得，這樣的代碼設計有沒有問題。

代碼是某位學員在項目中遇到的，我拿過來給簡化了一下。

其實這樣的代碼可能會導致內存泄漏，如果第一個 malloc 申請成功，第二個 malloc 申請失敗，直接 return -1 退出函數，就會導致前面申請的內存沒有釋放，長期運行，極有可能導致安全隱患。

對于這種連續初始化的代碼，一旦某個初始化失敗，應該要考慮到如何釋放前面已經申請的資源。

最簡單的方法，就是在 return 退出之前，把前面的資源清理掉。

不過這種方法，越往下初始化越多，退出的時候需要釋放的也就越多，重復的代碼也會很多。

或者使用 do while 語句，把代碼放在循環里面，初始化失敗后，直接跳出循環，不需要再進行下面的初始化，然后統一在某個地方釋放資源。

類似的還有 goto 語句，也是把釋放資源統一處理。

大家還有沒有其他更好的辦法，如果你也遇到過，歡迎在評論區交流。