NULL-pointer dereference fix and wrapper for free()ing allocated paths

mlabbe / nativefiledialog

A tiny, neat C library that portably invokes native file open and save dialogs.

zlib License

1.77k stars 209 forks source link

Closed szreder closed 3 years ago

szreder commented 4 years ago

Commit 1728cb7da25b61e1bbb196eec53700aa679a029b moves memcpy() further down below to avoid passing a potential NULL pointer after a failed memory allocation.
Commit 8bdd02dd6160843f1368e39e892f2f3ce4dee7a4 adds a wrapper for releasing memory of allocated paths. The paths are allocated with NFDi_Malloc(), so release them with NFDi_Free() for consistency.

Tested on 64-bit Linux (GCC and Clang), 64-bit Windows 10 (Visual Studio).

mlabbe commented 3 years ago

Fixed in devel because the code in question was refactored.