[asterisk-bugs] [JIRA] (ASTERISK-27726) chan_mobile presents incorrect inbound Caller-ID names

[Brian (JIRA)]

     [ https://issues.asterisk.org/jira/browse/ASTERISK-27726?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Brian updated ASTERISK-27726:
-----------------------------

    Description: 
chan_mobile requires that each cell phone be defined in chan_mobile.conf with unique name (e.g. "iphone").  I observed that when presenting inbound caller ID information, chan_mobile always presents the name of phone that received the call instead of the name of the caller.  So a call from "John Smith" at 505-555-5555 is presented as a call from "iphone" at 505-555-5555.

Looking into the source code, I see that hfp_parse_clip parses out the caller ID number, but does not attempt to parse out the optional caller ID name.  

Later in the code, the ast_channel routine calls ast_channel_alloc to create the channel.  When it does this it provides the cell phone name (e.g. “iphone”) as the fourth parameter, where the caller ID name should really belong (or a null string if caller ID name is not provided, so the dial plan can use alternate databases to retrieve a name).

These portions of the code have not changed in versions 14.x or 15.x, so the problem presumably also appears there, though I don't have an environment to test with.

This is an initial issue report.  I intend to develop a patch that I will submit for review through Gerrit.

  was:
chan_mobile requires that each cell phone be defined in chan_mobile.conf with unique name (e.g. "iphone").  I observed that when presenting inbound caller ID information, chan_mobile always presents the name of phone that received the call instead of the name of the caller.  So a call from "John Smith" at 505-555-5555 is presented as a call from "iphone" at 505-555-5555.

Looking into the source code, I see that hfp_parse_clip parses out the caller ID number, but does not attempt to parse out the optional caller ID name.  

Later in the code, the ast_channel routine calls ast_channel_alloc to create the channel.  When it does this it provides the cell phone name (e.g. “iphone”) as the fourth parameter, where the caller ID name should really belong (or a null string if caller ID name is not provided, so the dial plan can use alternate databases to retrieve a name).

These portions of the code have not changed in versions 14.x or 15.x, so the problem presumably also appears there, though I don't have an environment to test with.


> chan_mobile presents incorrect inbound Caller-ID names
> ------------------------------------------------------
>
>                 Key: ASTERISK-27726
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-27726
>             Project: Asterisk
>          Issue Type: Bug
>      Security Level: None
>          Components: Addons/chan_mobile
>    Affects Versions: 13.19.1, 14.7.6, 15.2.2
>            Reporter: Brian
>            Severity: Minor
>
> chan_mobile requires that each cell phone be defined in chan_mobile.conf with unique name (e.g. "iphone").  I observed that when presenting inbound caller ID information, chan_mobile always presents the name of phone that received the call instead of the name of the caller.  So a call from "John Smith" at 505-555-5555 is presented as a call from "iphone" at 505-555-5555.
> Looking into the source code, I see that hfp_parse_clip parses out the caller ID number, but does not attempt to parse out the optional caller ID name.  
> Later in the code, the ast_channel routine calls ast_channel_alloc to create the channel.  When it does this it provides the cell phone name (e.g. “iphone”) as the fourth parameter, where the caller ID name should really belong (or a null string if caller ID name is not provided, so the dial plan can use alternate databases to retrieve a name).
> These portions of the code have not changed in versions 14.x or 15.x, so the problem presumably also appears there, though I don't have an environment to test with.
> This is an initial issue report.  I intend to develop a patch that I will submit for review through Gerrit.



--
This message was sent by Atlassian JIRA
(v6.2#6252)