Skip to content

Commit 765ac20

Browse files
Dmitry Bogatovflub
Dmitry Bogatov
authored and
flub
committed
Change type of dc_msg_t.text to String
Also, remove `send-garbage' command from REPL, since it is not possible to send non-utf8 string anymore.
1 parent d4650ba commit 765ac20

File tree

11 files changed

+117
-124
lines changed

11 files changed

+117
-124
lines changed

‎deltachat-ffi/src/lib.rs‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -374,6 +374,7 @@ pub unsafe extern "C" fn dc_send_text_msg(
374374
) -> u32 {
375375
assert!(!context.is_null());
376376
let context = &*context;
377+
let text_to_send = dc_tools::to_string_lossy(text_to_send);
377378

378379
dc_chat::dc_send_text_msg(context, chat_id, text_to_send)
379380
}

‎examples/repl/cmdline.rs‎

Lines changed: 3 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -914,30 +914,17 @@ pub unsafe fn dc_cmdline(context: &Context, line: &str) -> Result<(), failure::E
914914
ensure!(!sel_chat.is_null(), "No chat selected.");
915915
ensure!(!arg1.is_empty(), "No message text given.");
916916

917-
let msg = CString::yolo(format!("{} {}", arg1, arg2));
917+
let msg = format!("{} {}", arg1, arg2);
918918

919-
if 0 != dc_send_text_msg(context, dc_chat_get_id(sel_chat), msg.as_ptr()) {
919+
if 0 != dc_send_text_msg(context, dc_chat_get_id(sel_chat), msg) {
920920
println!("Message sent.");
921921
} else {
922922
bail!("Sending failed.");
923923
}
924924
}
925-
"send-garbage" => {
926-
ensure!(!sel_chat.is_null(), "No chat selected.");
927-
let msg = b"\xff\x00"; // NUL-terminated C-string, that is malformed utf-8
928-
if 0 != dc_send_text_msg(context, dc_chat_get_id(sel_chat), msg.as_ptr().cast()) {
929-
println!("Malformed utf-8 succesfully send. Not nice.");
930-
} else {
931-
bail!("Garbage sending failed, as expected.");
932-
}
933-
}
934925
"sendempty" => {
935926
ensure!(!sel_chat.is_null(), "No chat selected.");
936-
if 0 != dc_send_text_msg(
937-
context,
938-
dc_chat_get_id(sel_chat),
939-
b"\x00" as *const u8 as *const libc::c_char,
940-
) {
927+
if 0 != dc_send_text_msg(context, dc_chat_get_id(sel_chat), "".into()) {
941928
println!("Message sent.");
942929
} else {
943930
bail!("Sending failed.");

‎examples/repl/main.rs‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -291,7 +291,7 @@ const DB_COMMANDS: [&'static str; 11] = [
291291
"housekeeping",
292292
];
293293

294-
const CHAT_COMMANDS: [&'static str; 25] = [
294+
const CHAT_COMMANDS: [&'static str; 24] = [
295295
"listchats",
296296
"listarchived",
297297
"chat",
@@ -309,7 +309,6 @@ const CHAT_COMMANDS: [&'static str; 25] = [
309309
"dellocations",
310310
"getlocations",
311311
"send",
312-
"send-garbage",
313312
"sendimage",
314313
"sendfile",
315314
"draft",

‎examples/simple.rs‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -97,8 +97,7 @@ fn main() {
9797
println!("sending a message");
9898
let contact_id = dc_create_contact(&ctx, std::ptr::null(), email.as_ptr());
9999
let chat_id = dc_create_chat_by_contact_id(&ctx, contact_id);
100-
let msg_text = CString::new("Hi, here is my first message!").unwrap();
101-
dc_send_text_msg(&ctx, chat_id, msg_text.as_ptr());
100+
dc_send_text_msg(&ctx, chat_id, "Hi, here is my first message!".into());
102101

103102
println!("fetching chats..");
104103
let chats = Chatlist::try_load(&ctx, 0, None, None).unwrap();

‎src/dc_chat.rs‎

Lines changed: 39 additions & 65 deletions
Original file line numberDiff line numberDiff line change
@@ -725,7 +725,7 @@ unsafe fn prepare_msg_raw(
725725
timestamp,
726726
(*msg).type_0,
727727
(*msg).state,
728-
if !(*msg).text.is_null() { Some(as_str((*msg).text)) } else { None },
728+
(*msg).text,
729729
(*msg).param.to_string(),
730730
(*msg).hidden,
731731
to_string(new_in_reply_to),
@@ -951,7 +951,7 @@ pub unsafe fn dc_send_msg<'a>(
951951
pub unsafe fn dc_send_text_msg(
952952
context: &Context,
953953
chat_id: uint32_t,
954-
text_to_send: *const libc::c_char,
954+
text_to_send: String,
955955
) -> uint32_t {
956956
if chat_id <= 9 {
957957
warn!(
@@ -961,18 +961,8 @@ pub unsafe fn dc_send_text_msg(
961961
return 0;
962962
}
963963

964-
if text_to_send.is_null() {
965-
warn!(context, 0, "dc_send_text_msg: text_to_send is emtpy");
966-
return 0;
967-
}
968-
969-
if let Err(err) = as_str_safe(text_to_send) {
970-
warn!(context, 0, "{}", err);
971-
return 0;
972-
}
973-
974964
let mut msg = dc_msg_new(context, Viewtype::Text);
975-
(*msg).text = dc_strdup(text_to_send);
965+
(*msg).text = Some(text_to_send);
976966
let ret = dc_send_msg(context, chat_id, msg);
977967
dc_msg_unref(msg);
978968
ret
@@ -1002,9 +992,7 @@ unsafe fn set_draft_raw(context: &Context, chat_id: uint32_t, msg: *mut dc_msg_t
1002992
// save new draft
1003993
if !msg.is_null() {
1004994
if (*msg).type_0 == Viewtype::Text {
1005-
if (*msg).text.is_null() || *(*msg).text.offset(0isize) as libc::c_int == 0i32 {
1006-
OK_TO_CONTINUE = false;
1007-
}
995+
OK_TO_CONTINUE = (*msg).text.as_ref().map_or(false, |s| !s.is_empty());
1008996
} else if msgtype_has_file((*msg).type_0) {
1009997
let mut pathNfilename = (*msg)
1010998
.param
@@ -1037,11 +1025,7 @@ unsafe fn set_draft_raw(context: &Context, chat_id: uint32_t, msg: *mut dc_msg_t
10371025
time(),
10381026
(*msg).type_0,
10391027
DC_STATE_OUT_DRAFT,
1040-
if !(*msg).text.is_null() {
1041-
as_str((*msg).text)
1042-
} else {
1043-
""
1044-
},
1028+
(*msg).text.deref().unwrap_or(""),
10451029
(*msg).param.to_string(),
10461030
1,
10471031
],
@@ -1617,14 +1601,12 @@ pub unsafe fn dc_add_contact_to_chat_ex(
16171601
if OK_TO_CONTINUE {
16181602
if (*chat).param.get_int(Param::Unpromoted).unwrap_or_default() == 0 {
16191603
(*msg).type_0 = Viewtype::Text;
1620-
(*msg).text = context
1621-
.stock_system_msg(
1622-
StockMessage::MsgAddMember,
1623-
as_str((*contact).addr),
1624-
"",
1625-
DC_CONTACT_ID_SELF as uint32_t,
1626-
)
1627-
.strdup();
1604+
(*msg).text = Some(context.stock_system_msg(
1605+
StockMessage::MsgAddMember,
1606+
as_str((*contact).addr),
1607+
"",
1608+
DC_CONTACT_ID_SELF as uint32_t,
1609+
));
16281610
(*msg).param.set_int(Param::Cmd, 4);
16291611
if !(*contact).addr.is_null() {
16301612
(*msg).param.set(Param::Arg, as_str((*contact).addr));
@@ -1732,23 +1714,19 @@ pub unsafe fn dc_remove_contact_from_chat(
17321714
(*msg).type_0 = Viewtype::Text;
17331715
if (*contact).id == 1 as libc::c_uint {
17341716
dc_set_group_explicitly_left(context, (*chat).grpid);
1735-
(*msg).text = context
1736-
.stock_system_msg(
1737-
StockMessage::MsgGroupLeft,
1738-
"",
1739-
"",
1740-
DC_CONTACT_ID_SELF as u32,
1741-
)
1742-
.strdup();
1717+
(*msg).text = Some(context.stock_system_msg(
1718+
StockMessage::MsgGroupLeft,
1719+
"",
1720+
"",
1721+
DC_CONTACT_ID_SELF as u32,
1722+
));
17431723
} else {
1744-
(*msg).text = context
1745-
.stock_system_msg(
1746-
StockMessage::MsgDelMember,
1747-
as_str((*contact).addr),
1748-
"",
1749-
DC_CONTACT_ID_SELF as u32,
1750-
)
1751-
.strdup();
1724+
(*msg).text = Some(context.stock_system_msg(
1725+
StockMessage::MsgDelMember,
1726+
as_str((*contact).addr),
1727+
"",
1728+
DC_CONTACT_ID_SELF as u32,
1729+
));
17521730
}
17531731
(*msg).param.set_int(Param::Cmd, 5);
17541732
if !(*contact).addr.is_null() {
@@ -1849,14 +1827,12 @@ pub unsafe fn dc_set_chat_name(
18491827
{
18501828
if (*chat).param.get_int(Param::Unpromoted).unwrap_or_default() == 0 {
18511829
(*msg).type_0 = Viewtype::Text;
1852-
(*msg).text = context
1853-
.stock_system_msg(
1854-
StockMessage::MsgGrpName,
1855-
as_str((*chat).name),
1856-
as_str(new_name),
1857-
DC_CONTACT_ID_SELF as u32,
1858-
)
1859-
.strdup();
1830+
(*msg).text = Some(context.stock_system_msg(
1831+
StockMessage::MsgGrpName,
1832+
as_str((*chat).name),
1833+
as_str(new_name),
1834+
DC_CONTACT_ID_SELF as u32,
1835+
));
18601836
(*msg).param.set_int(Param::Cmd, 2);
18611837
if !(*chat).name.is_null() {
18621838
(*msg).param.set(Param::Arg, as_str((*chat).name));
@@ -1923,18 +1899,16 @@ pub unsafe fn dc_set_chat_profile_image(
19231899
(*msg).param.set_int(Param::Cmd, 3);
19241900
(*msg).param.set(Param::Arg, as_str(new_image_rel));
19251901
(*msg).type_0 = Viewtype::Text;
1926-
(*msg).text = context
1927-
.stock_system_msg(
1928-
if !new_image_rel.is_null() {
1929-
StockMessage::MsgGrpImgChanged
1930-
} else {
1931-
StockMessage::MsgGrpImgDeleted
1932-
},
1933-
"",
1934-
"",
1935-
DC_CONTACT_ID_SELF as uint32_t,
1936-
)
1937-
.strdup();
1902+
(*msg).text = Some(context.stock_system_msg(
1903+
if !new_image_rel.is_null() {
1904+
StockMessage::MsgGrpImgChanged
1905+
} else {
1906+
StockMessage::MsgGrpImgDeleted
1907+
},
1908+
"",
1909+
"",
1910+
DC_CONTACT_ID_SELF as uint32_t,
1911+
));
19381912
(*msg).id = dc_send_msg(context, chat_id, msg);
19391913
context.call_cb(
19401914
Event::MSGS_CHANGED,

‎src/dc_location.rs‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -100,9 +100,8 @@ pub unsafe fn dc_send_locations_to_chat(
100100
{
101101
if 0 != seconds && !is_sending_locations_before {
102102
msg = dc_msg_new(context, Viewtype::Text);
103-
(*msg).text = context
104-
.stock_system_msg(StockMessage::MsgLocationEnabled, "", "", 0)
105-
.strdup();
103+
(*msg).text =
104+
Some(context.stock_system_msg(StockMessage::MsgLocationEnabled, "", "", 0));
106105
(*msg).param.set_int(Param::Cmd, 8);
107106
dc_send_msg(context, chat_id, msg);
108107
} else if 0 == seconds && is_sending_locations_before {

‎src/dc_lot.rs‎

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ use crate::dc_tools::*;
66
use crate::stock::StockMessage;
77
use crate::types::*;
88
use crate::x::*;
9+
use std::ffi::CString;
10+
use std::ptr;
911

1012
/* * Structure behind dc_lot_t */
1113
#[derive(Copy, Clone)]
@@ -160,8 +162,16 @@ pub unsafe fn dc_lot_fill(
160162
(*lot).text1_meaning = 2i32
161163
}
162164
}
165+
166+
let msgtext_c = (*msg)
167+
.text
168+
.as_ref()
169+
.map(|s| CString::yolo(String::as_str(s)));
170+
let msgtext_ptr = msgtext_c.map_or(ptr::null(), |s| s.as_ptr());
171+
163172
(*lot).text2 =
164-
dc_msg_get_summarytext_by_raw((*msg).type_0, (*msg).text, &mut (*msg).param, 160, context);
173+
dc_msg_get_summarytext_by_raw((*msg).type_0, msgtext_ptr, &mut (*msg).param, 160, context);
174+
165175
(*lot).timestamp = dc_msg_get_timestamp(msg);
166176
(*lot).state = (*msg).state;
167177
}

‎src/dc_mimefactory.rs‎

Lines changed: 25 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ use mmime::mailmime_types_helper::*;
99
use mmime::mailmime_write_mem::*;
1010
use mmime::mmapstring::*;
1111
use mmime::other::*;
12+
use std::ptr;
1213

1314
use crate::constants::*;
1415
use crate::context::Context;
@@ -799,12 +800,17 @@ pub unsafe fn dc_mimefactory_render(mut factory: *mut dc_mimefactory_t) -> libc:
799800
)
800801
}
801802

802-
let mut final_text: *const libc::c_char = 0 as *const libc::c_char;
803-
if !placeholdertext.is_null() {
804-
final_text = placeholdertext
805-
} else if !(*msg).text.is_null() && 0 != *(*msg).text.offset(0isize) as libc::c_int {
806-
final_text = (*msg).text
807-
}
803+
let final_text = {
804+
if !placeholdertext.is_null() {
805+
to_string(placeholdertext)
806+
} else if let Some(ref text) = (*msg).text {
807+
text.clone()
808+
} else {
809+
"".into()
810+
}
811+
};
812+
let final_text = CString::yolo(final_text);
813+
808814
let footer: *mut libc::c_char = (*factory).selfstatus;
809815
message_text = dc_mprintf(
810816
b"%s%s%s%s%s\x00" as *const u8 as *const libc::c_char,
@@ -813,12 +819,8 @@ pub unsafe fn dc_mimefactory_render(mut factory: *mut dc_mimefactory_t) -> libc:
813819
} else {
814820
b"\x00" as *const u8 as *const libc::c_char
815821
},
816-
if !final_text.is_null() {
817-
final_text
818-
} else {
819-
b"\x00" as *const u8 as *const libc::c_char
820-
},
821-
if !final_text.is_null()
822+
final_text.as_ptr(),
823+
if final_text != CString::yolo("")
822824
&& !footer.is_null()
823825
&& 0 != *footer.offset(0isize) as libc::c_int
824826
{
@@ -1094,8 +1096,17 @@ unsafe fn get_subject(
10941096
) -> *mut libc::c_char {
10951097
let context = (*chat).context;
10961098
let ret: *mut libc::c_char;
1097-
let raw_subject =
1098-
dc_msg_get_summarytext_by_raw((*msg).type_0, (*msg).text, &mut (*msg).param, 32, context);
1099+
1100+
let raw_subject = {
1101+
let msgtext_c = (*msg)
1102+
.text
1103+
.as_ref()
1104+
.map(|s| CString::yolo(String::as_str(s)));
1105+
let msgtext_ptr = msgtext_c.map_or(ptr::null(), |s| s.as_ptr());
1106+
1107+
dc_msg_get_summarytext_by_raw((*msg).type_0, msgtext_ptr, &mut (*msg).param, 32, context)
1108+
};
1109+
10991110
let fwd = if 0 != afwd_email {
11001111
b"Fwd: \x00" as *const u8 as *const libc::c_char
11011112
} else {

0 commit comments

Comments
 (0)