[asterisk-bugs] [Asterisk 0014531]: MWI subscriptions does not works if there is no HINT for extension

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14531 
====================================================================== 
Reported By:                festr
Assigned To:                file
====================================================================== 
Project:                    Asterisk
Issue ID:                   14531
Category:                   Channels/chan_sip/Subscriptions
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 177786 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-02-23 09:44 CST
Last Modified:              2009-02-24 09:20 CST
====================================================================== 
Summary:                    MWI subscriptions does not works if there is no HINT
for extension
Description: 
in chan_sip.c: inside function handle_request_subscribe there is "gotdest =
get_destination(p, NULL)" which is searching for hints and if does not find
any it will end. But if SUBSCRIBE event is "message-summary" for voicemail
checking (MWI), it will never reach this code if there is no hints defined
(for mwi there is no need to define any hint). This leads to non working
MWI at all. 

I've tried commenting out check for gotdest to proove that mwi is working
without defined hints. But I dont how to make proper patch :)
====================================================================== 

---------------------------------------------------------------------- 
 (0100621) svnbot (reporter) - 2009-02-24 09:20
 http://bugs.digium.com/view.php?id=14531#c100621 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 178224

_U  branches/1.6.0/
U   branches/1.6.0/channels/chan_sip.c

------------------------------------------------------------------------
r178224 | file | 2009-02-24 09:20:55 -0600 (Tue, 24 Feb 2009) | 23 lines

Merged revisions 178213 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
  r178213 | file | 2009-02-24 11:18:38 -0400 (Tue, 24 Feb 2009) | 16 lines
  
  Merged revisions 178205 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r178205 | file | 2009-02-24 11:16:07 -0400 (Tue, 24 Feb 2009) | 9
lines
    
    Skip check for extension when subscribing for MWI.
    
    Since the remote side is not actually subscribing to a specific
extension when
    subscribing for MWI just skip the check to see if the extension
exists. They can't use it
    to specify the mailbox either since we require configuration of that
in sip.conf
    
    (closes issue http://bugs.digium.com/view.php?id=14531)
    Reported by: festr
  ........
................

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=178224 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-02-24 09:20 svnbot         Checkin                                      
2009-02-24 09:20 svnbot         Note Added: 0100621                          
======================================================================