ews: avoid nullptr deref processing <t:AlternateId>

jengelh · jengelh · commit 76c3920e0ce5 · 2025-01-05T22:46:30.000+01:00
Fixes: gromox-2.23-10-g9ff3922c7
References: GXH-123
diff --git a/exch/ews/serialization.cpp b/exch/ews/serialization.cpp
@@ -1472,7 +1472,8 @@ void mConvertIdResponseMessage::serialize(tinyxml2::XMLElement* xml) const
 {
 	mResponseMessageType::serialize(xml);
 	tinyxml2::XMLElement* altid = XMLDUMPM(AlternateId);
-	altid->SetAttribute("xsi:type", "t:AlternateIdType");
+	if (altid != nullptr)
+		altid->SetAttribute("xsi:type", "t:AlternateIdType");
 }
 
 mCopyFolderRequest::mCopyFolderRequest(const tinyxml2::XMLElement* xml) :

Original file line number	Diff line number	Diff line change
`@@ -1472,7 +1472,8 @@ void mConvertIdResponseMessage::serialize(tinyxml2::XMLElement* xml) const`
`1472`	`1472`	`{`
`1473`	`1473`	`mResponseMessageType::serialize(xml);`
`1474`	`1474`	`tinyxml2::XMLElement* altid = XMLDUMPM(AlternateId);`
`1475`		`- altid->SetAttribute("xsi:type", "t:AlternateIdType");`
	`1475`	`+ if (altid != nullptr)`
	`1476`	`+ altid->SetAttribute("xsi:type", "t:AlternateIdType");`
`1476`	`1477`	`}`
`1477`	`1478`
`1478`	`1479`	`mCopyFolderRequest::mCopyFolderRequest(const tinyxml2::XMLElement* xml) :`