"ret->cd_direct" is never set before. It is set in the "if" upperwards but
then the function is terminated with "return".
}
int smb_iconv_t_destructor(smb_iconv_t hwd)
-{
+{
#ifdef HAVE_NATIVE_ICONV
if (hwd->cd_pull != NULL && hwd->cd_pull != (iconv_t)-1)
iconv_close(hwd->cd_pull);
}
if (is_utf16(tocode)) {
ret->direct = sys_iconv;
- /* could be set just above - so we need to close iconv */
- if (ret->cd_direct != NULL && ret->cd_direct != (iconv_t)-1)
- iconv_close(ret->cd_direct);
ret->cd_direct = ret->cd_pull;
ret->cd_pull = NULL;
return ret;