oxcmail: repair nullptr deref when run under test environment

jengelh · jengelh · commit 66356f8dc2e3 · 2025-06-12T17:30:48.000+02:00
Tests do not necessarily set all the function pointers to something.

Fixes: gromox-2.26-91-gc585401b8
diff --git a/lib/mapi/oxcmail.cpp b/lib/mapi/oxcmail.cpp
@@ -231,7 +231,8 @@ BOOL oxcmail_username_to_entryid(const char *username,
 {
 	std::string essdn;
 
-	if (oxcmail_get_user_ids(username, nullptr, nullptr, dtpp) &&
+	if (oxcmail_get_user_ids != nullptr &&
+	    oxcmail_get_user_ids(username, nullptr, nullptr, dtpp) &&
 	    cvt_username_to_essdn(username, g_oxcmail_org_name,
 	    oxcmail_get_user_ids, oxcmail_get_domain_ids,
 	    essdn) == ecSuccess)

Original file line number	Diff line number	Diff line change
`@@ -231,7 +231,8 @@ BOOL oxcmail_username_to_entryid(const char *username,`
`231`	`231`	`{`
`232`	`232`	`std::string essdn;`
`233`	`233`
`234`		`- if (oxcmail_get_user_ids(username, nullptr, nullptr, dtpp) &&`
	`234`	`+ if (oxcmail_get_user_ids != nullptr &&`
	`235`	`+ oxcmail_get_user_ids(username, nullptr, nullptr, dtpp) &&`
`235`	`236`	`cvt_username_to_essdn(username, g_oxcmail_org_name,`
`236`	`237`	`oxcmail_get_user_ids, oxcmail_get_domain_ids,`
`237`	`238`	`essdn) == ecSuccess)`