[Asterisk-code-review] app voicemail: Add Mailbox Aliases (asterisk[16])

[Benjamin Keith Ford]

Benjamin Keith Ford has posted comments on this change. ( https://gerrit.asterisk.org/10800 )

Change subject: app_voicemail:  Add Mailbox Aliases
......................................................................


Patch Set 1:

(3 comments)

https://gerrit.asterisk.org/#/c/10800/1/CHANGES
File CHANGES:

https://gerrit.asterisk.org/#/c/10800/1/CHANGES@18
PS1, Line 18:    which will allow you to create aliases for physical mailboxes. 
Red space, just ignore if other comment doesn't require action :)


https://gerrit.asterisk.org/#/c/10800/1/apps/app_voicemail.c
File apps/app_voicemail.c:

https://gerrit.asterisk.org/#/c/10800/1/apps/app_voicemail.c@1804
PS1, Line 1804: 		mapping = ao2_find(alias_mailbox_mappings, search_string, OBJ_SEARCH_KEY);
              : 		if (mapping) {
              : 			char *search_mailbox;
              : 			char *search_context;
              : 
              : 			separate_mailbox(ast_strdupa(mapping->mailbox), &search_mailbox, &search_context);
              : 			vmu = find_user(ivm, search_mailbox, search_context);
              : 		}
Should there be an ao2_ref call here? Just curious since it would follow the pattern of the other calls like this.


https://gerrit.asterisk.org/#/c/10800/1/apps/app_voicemail.c@8157
PS1, Line 8157: 		ast_publish_mwi_state_channel(mailbox, context, new + urgent, old, channel_id);
Would this potentially post a state update twice (or more than twice) for a mailbox, specifically ones that can be found but also have aliases for them?



-- 
To view, visit https://gerrit.asterisk.org/10800
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-MessageType: comment
Gerrit-Change-Id: I395b9205c91523a334fe971be0d1de4522067b04
Gerrit-Change-Number: 10800
Gerrit-PatchSet: 1
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Friendly Automation (1000185)
Gerrit-Comment-Date: Thu, 13 Dec 2018 17:48:06 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20181213/3fcd4d42/attachment.html>