Skip to content

Conversation

@umgfoin
Copy link
Contributor

@umgfoin umgfoin commented Nov 2, 2025

ONEOFF_ENTRYID tmp_entryis never assigned a valid vmime:mailbox-parsed email_addr.

ONEOFF_ENTRYID::pmail_address and -::pdisplay_name-members need to be assigned not earlier than email_addr received a valid entry by vmime:

email_addr.set(*mb);
tmp_entry.pmail_address = email_addr.addr;
tmp_entry.pdisplay_name = email_addr.display_name;

Fixes #185

@umgfoin umgfoin force-pushed the PR_umgfoin_fix_reply-to-parsing branch 3 times, most recently from c251888 to 72935b3 Compare November 3, 2025 16:46
[In caa9793, ``struct ONEOFF_ENTRYID`` was changed from storing raw pointers
to storing ``std::string``. This means that ``tmp_entryid.pdisplay_name = x``
now takes a copy, and that statement was happening too early. –jengelh]

Fixes: gromox-3.0-76-gcaa979304
References: GXH-185
Signed-off-by: umgfoin <[email protected]>
@jengelh jengelh force-pushed the PR_umgfoin_fix_reply-to-parsing branch from 72935b3 to 72155c6 Compare November 3, 2025 18:36
@jengelh jengelh merged commit 72155c6 into grommunio:master Nov 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants