[asterisk-bugs] [Asterisk 0015075]: [patch] chan_mobile.c:1038 mbl_read: read error 107

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15075 
====================================================================== 
Reported By:                hmld
Assigned To:                mnicholson
====================================================================== 
Project:                    Asterisk
Issue ID:                   15075
Category:                   Addons/chan_mobile
Reproducibility:            always
Severity:                   tweak
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 905 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-05-10 18:58 CDT
Last Modified:              2009-05-27 12:19 CDT
====================================================================== 
Summary:                    [patch] chan_mobile.c:1038 mbl_read: read error 107
Description: 
In bug report http://bugs.digium.com/view.php?id=15037 I mentioned that a
"read error 104" happened repeatedly. Now, with chan_mobile version 905 and
the quickly evolving git version of the bluetooth kernel subsystem, when
asterisk is making a call via the cell phone I'm getting read error 107. 
This error number translates to "transport endpoint is not connected"
(formely: 104 "connection reset by peer"). But voice is back and clear.

Error 107 stops to be printed two seconds and 80856 identical copies later
the very moment the mobile reports that it is ringing the other side :

- Executing [0xxxxxxxxx at phones:1] Dial("SIP/1000-b9660308",
"Mobile/Nokia/0xxxxxxxx,45") in new stack
    -- Called Nokia/0xxxxxxxxx
[May 11 18:32:45] ERROR[7791]: chan_mobile.c:1038 mbl_read: read error
104
[...] 
[May 11 18:32:47] ERROR[7791]: chan_mobile.c:1038 mbl_read: read error
107
    -- Mobile/Nokia-1728 is ringing
    -- Mobile/Nokia-1728 is ringing
  == Spawn extension (phones, 0xxxxxxxxx, 1) exited non-zero on
'SIP/1000-b9660308'
    -- Bluetooth Device Nokia has disconnected.
    -- Bluetooth Device Nokia has connected, initilizing...
    -- Bluetooth Device Nokia initialized and ready.


The high poll frequency (about 40 kHz) points to a problem with file
descriptor handling around the select system call. Possibly FD_CLR is left
out on in case of a read error?



====================================================================== 

---------------------------------------------------------------------- 
 (0105561) svnbot (reporter) - 2009-05-27 12:19
 https://issues.asterisk.org/view.php?id=15075#c105561 
---------------------------------------------------------------------- 
Repository: asterisk-addons
Revision: 928

_U  branches/1.6.2/
U   branches/1.6.2/channels/chan_mobile.c

------------------------------------------------------------------------
r928 | mnicholson | 2009-05-27 12:19:36 -0500 (Wed, 27 May 2009) | 27
lines

Merged revisions 926-927 via svnmerge from 
https://origsvn.digium.com/svn/asterisk-addons/trunk

........
  r926 | mnicholson | 2009-05-27 12:07:00 -0500 (Wed, 27 May 2009) | 9
lines
  
  Lock the pvt structure in the mbl_read() and mbl_write() functions.
  
  (issue https://issues.asterisk.org/view.php?id=15075)
  Reported by: hmld
  Patches:
        chan_mobile_v2.diff uploaded by zvision (license 798)
        chan_mobile_v2a.diff uploaded by hmld (license 777)
  Tested by: hmld, zvision, rseste
........
  r927 | mnicholson | 2009-05-27 12:13:44 -0500 (Wed, 27 May 2009) | 9
lines
  
  Handle read errors from the sco_socket by closing the socket conenction
and waiting for another one before reading again.  This is non standard
behavior to accomidate certain Nokia handsets.
  
  (closes issue https://issues.asterisk.org/view.php?id=15075)
  Reported by: hmld
  Patches:
        chan_mobile_v2.diff uploaded by zvision (license 798)
        chan_mobile_v2a.diff uploaded by hmld (license 777)
  Tested by: hmld, zvision, rseste
........

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

http://svn.digium.com/view/asterisk-addons?view=rev&revision=928 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-05-27 12:19 svnbot         Checkin                                      
2009-05-27 12:19 svnbot         Note Added: 0105561                          
======================================================================