Bugfix: returned success (true) even if smb_addmsghdr() failed.

Bugfix: didn't free data fields if smb_addmsghdr() failed.

src/sbbs3/qwktomsg.cpp

@@ -52,6 +52,7 @@ bool sbbs_t::qwktomsg(FILE *qwk_fp, char *hdrblk, char fromhub, uint subnum
 	ushort	xlat;
 	long	l,bodylen,taillen,length;
 	bool	header_cont=false;
+	bool	success=true;
 	ulong	crc,block,blocks;
 	smbmsg_t	msg;
 	smbmsg_t	remsg;
@@ -508,13 +509,16 @@ bool sbbs_t::qwktomsg(FILE *qwk_fp, char *hdrblk, char fromhub, uint subnum
 	}
 	fflush(smb.sdt_fp);
 
-	if((i=smb_addmsghdr(&smb,&msg,storage))!=0)	// calls smb_unlocksmbhdr() 
-		errormsg(WHERE,ERR_WRITE,smb.file,i);
+	if((i=smb_addmsghdr(&smb,&msg,storage))!=SMB_SUCCESS) {	// calls smb_unlocksmbhdr() 
+		errormsg(WHERE,ERR_WRITE,smb.file,i,smb.last_error);
+		smb_freemsg_dfields(&smb,&msg,1);
+		success=false;
+	}
 
 	smb_freemsgmem(&msg);
 
 	LFREE(body);
 	LFREE(tail);
 
-	return(true);
+	return(success);
 }