Check returned value of fopen() #9775

util.cpp#L216 if fopen failed, a null file pointer could be passed to FileWriteStr(vMsgsBeforeOpenLog->front(), fileout);

As I've understood the problem, the solution would be simply passing a "NULL" to FileWriteStr as the second argument and an empty string as the first argument. Is that correct or did you have something else in mind?

I'd say the most straightforward and correct solution is to put everything until the delete vMsgsBeforeOpenLog under the if (fileout) conditional.