Syping/libragephoto
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

fix potential iconv memleak

Browse source
This commit is contained in:
Syping 2021-10-20 08:53:55 +02:00
parent e996f4399d
commit d71f3641e2
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/RagePhoto.cpp
View file

@ -467,11 +467,11 @@ bool RagePhoto::save(char *data, PhotoFormat photoFormat)
char *src = photoString; char *src = photoString;
char *dst = photoHeader; char *dst = photoHeader;
const size_t ret = iconv(iconv_in, &src, &src_s, &dst, &dst_s); const size_t ret = iconv(iconv_in, &src, &src_s, &dst, &dst_s);
iconv_close(iconv_in);
if (ret == static_cast<size_t>(-1)) { if (ret == static_cast<size_t>(-1)) {
p_error = Error::UnicodeHeaderError; // 5 p_error = Error::UnicodeHeaderError; // 5
return false; return false;
} }
iconv_close(iconv_in);
const size_t photoHeader_size = p_photoString.size() * 2; const size_t photoHeader_size = p_photoString.size() * 2;
if (photoHeader_size > 256) { if (photoHeader_size > 256) {
p_error = Error::HeaderBufferTight; // 34 p_error = Error::HeaderBufferTight; // 34