(jikstra, hpk) fix a logic bug introduced with the stock-string merge which set the better message only if it was empty

This commit is contained in:
holger krekel
2019-08-08 19:21:52 +02:00
parent a41204b275
commit 447d3c3416
2 changed files with 3 additions and 6 deletions

View File

@@ -1212,11 +1212,8 @@ unsafe fn dc_mimeparser_add_single_part_if_known(
info!(
mimeparser.context,
0,
"Simpliifed text: \n---\n{}\n--\n",
String::from_utf8_lossy(std::slice::from_raw_parts(
decoded_data as *const u8,
decoded_data_bytes as usize,
))
"Simpliifed text: {}",
as_str(simplified_txt),
);
let mut part = dc_mimepart_new();

View File

@@ -1723,7 +1723,7 @@ unsafe fn check_verified_properties(
unsafe fn set_better_msg<T: AsRef<str>>(mime_parser: &mut dc_mimeparser_t, better_msg: T) {
let msg = better_msg.as_ref();
if !(msg.len() > 0) && !mime_parser.parts.is_empty() {
if msg.len() > 0 && !mime_parser.parts.is_empty() {
let part = &mut mime_parser.parts[0];
if (*part).type_0 == 10 {
free(part.msg as *mut libc::c_void);