diff options
author | Patrick C. McGinty <casey.mcginty@gmail.com> | 2011-03-08 00:18:52 -1000 |
---|---|---|
committer | Patrick C. McGinty <casey.mcginty@gmail.com> | 2011-03-08 00:45:27 -1000 |
commit | d81ae5a482050947169f10e89c4d45e982f6012f (patch) | |
tree | cf17a0bad9abebfa4f16065b7e70dfac0e9f2fa0 /easytag.spec | |
parent | a1f6f1f55c73959cad85d0fc23d5a00ad3326463 (diff) |
Stop accidental free of referenced FLAC vendor string
The libFLAC API is unclear about how it handles empty strings. The
original code assumed that empty string would be copied, when 'copy'
param of set_vendor_string was true, as indicated in the API docs.
Checking into the libFLAC source, when a string of length 0 is provided,
it will NOT be copied. Therfore, free'ing empty string will cause a
SIGSEGV or worse when the FLAC API tries to read the vendor string.
Diffstat (limited to 'easytag.spec')
0 files changed, 0 insertions, 0 deletions