summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2014-06-30hfpmodem: Fix crash with more than two callsSergio Checa Blanco1-1/+1
A periodic CLCC polling is started when there is an ongoing multiparty call and a new call appears in the system. A simple way to reproduce the crashing scenario is: 1. Place a call. 2. Place a second call. 3. Create a multiparty call with both calls. 4. Place a third call (incoming or outgoing does not matter). 5. Disconnect HFP from the modem. Within the function ciev_callheld_notify, the AT+CLCC command is also invoked, thus a new cyclic CLCC polling is started, and it overwrites the timer resource identifier stored in voicecall_data.clcc_source. This means that there are several timers doing the CLCC polling, but only one of those is under control, i.e. it can be removed through its source identifier, hence a timer source leak. This has a fatal consequence when the HFP modem is disconnected. The function hfp_voicecall_remove stops the timer that is under control before freeing the voicecall_data struct. However there are other timers that are still active and will execute its handler poll_clcc afterwards. Inside poll_clcc the driver_data is accessed, which is already NULL. A solution for this is to avoid starting a CLCC polling if there is already one active, i.e. clcc_source is not 0. By doing this the uncontrolled timers will not cycle forever.
2014-06-29Release 1.15Marcel Holtmann2-1/+12
2014-06-24gprs: provision the authentication methodPhilip Paeps1-0/+2
2014-06-24mbpi: add support for provisioning the auth methodPhilip Paeps1-0/+35
Use the authentication method from the mobile-broadband-provider-info database if it is specified and supported (we support CHAP and PAP). Default to CHAP if the database does not specify a method (i.e.: the previous behaviour).
2014-06-24include: add auth method to the provisioning APIPhilip Paeps1-0/+1
2014-06-21AUTHORS: Mention Philip's contributionsDenis Kenzior1-0/+1
2014-06-21atmodem: set the PPP authentication method to usePhilip Paeps1-0/+14
2014-06-21gprs: Make the code a bit easier to readDenis Kenzior1-3/+6
2014-06-21gprs: make PPP authentication method configurablePhilip Paeps1-0/+84
Add a new "AuthenticationMethod" property to select between "pap" and "chap" PPP authentication, defaulting to "chap" (i.e.: previous behaviour).
2014-06-21include: add auth method to the gprs context APIPhilip Paeps1-0/+6
2014-06-21doc: describe the AuthenticationMethod propertyPhilip Paeps1-0/+4
2014-06-21gatchat: Fix up minor coding style issuesDenis Kenzior2-1/+7
2014-06-21gatchat: implement PAP authenticationPhilip Paeps5-21/+247
Make the authentication method configurable, CHAP or PAP, defaulting to CHAP (i.e.: previous behaviour). Implementation details: o If PAP is configured, we NAK the CHAP authentication protocol option in LCP configuration requests and suggest PAP instead. This works around the amusing requirement of 3GPP TS 29.061 that modems must send a forced positive acknowledgement of the authentication method tried (i.e.: the modem will successfully accept any CHAP handshake, but if the network only supports PAP, the modem will hang up when it tries and fails to activate the PDP context) o The PAP Authenticate-Request is resent a hard-coded three times at ten-second intervals. This may be a bit too persistent. Chances are if it doesn't work the first time, it'll never work, but the RFC insists that we MUST retry.
2014-06-21AUTHORS: Mention Sergio's contributionsDenis Kenzior1-0/+1
2014-06-21hfpmodem: Split AT+VTS command into multiple +VTSSergio Checa Blanco1-3/+8
According to the standard "3GPP 27.007 v6.8.0" Appendix C.2.11, when sending multiple DTMF characters, these must go in individual +VTS commands for each tone. This adopts the AT modem approach. Before: AT+VTS=1234\r After: AT+VTS=1;+VTS=2;+VTS=3;+VTS=4\r
2014-06-10test: Adapt data test scripts to multi-modemAlfonso Sanchez-Beato2-47/+81
Some tests scripts were not ready to handle situations with more than one modem present. This change fixes the data context scripts.
2014-06-10test: Adapt voice call test scripts to multi-modemAlfonso Sanchez-Beato5-34/+75
Some tests scripts were not ready to handle situations with more than one modem present. This change fixes the scripts that handle voice calls.
2014-05-30hfp: Fix case where RING never arrivesDenis Kenzior1-1/+28
2014-05-21hfp: Use proper prefixDenis Kenzior1-1/+2
Otherwise the BCS command can consume unsolicited notifications. ofonod[3270]: < \r\n+BCS: 2\r\n ofonod[3270]: > AT+BCS=2\r ofonod[3270]: < \r\n+CIEV: 1,1\r\n ofonod[3270]: < \r\n+CIEV: 2,0\r\n ofonod[3270]: < \r\nOK\r\n
2014-04-22handsfree: correct crash on connect of hfpAndrew Earl1-1/+2
2014-04-10handsfree: Rename driver functionDenis Kenzior1-3/+3
To be more consistent with others in the same file
2014-04-10handsfree: Simplify logicDenis Kenzior1-18/+12
2014-04-10hfp: Add CNUM query to handsfree interface for subscriber numberAndrew Earl1-0/+88
2014-04-10handsfree: Rename variableDenis Kenzior1-6/+6
2014-04-10handsfree: Reflow logicDenis Kenzior1-7/+8
2014-04-10handsfree: Remove functionDenis Kenzior1-6/+1
No need for one line functions that are only used once
2014-04-10handsfree: Rename functionDenis Kenzior1-2/+2
No need for the ofono_ prefix here
2014-04-10handsfree: Remove unneeded variableDenis Kenzior1-2/+1
2014-04-10handsfree: Remove unneeded variable iDenis Kenzior1-2/+1
2014-04-10handsfree: style fixes for previous commitDenis Kenzior1-6/+7
2014-04-10hfp: Add subscriber number to handsfree propertiesAndrew Earl1-3/+121
2014-04-10include: Add subscriber number to handsfree APIAndrew Earl1-0/+6
2014-04-10doc: Add subscriber number to handsfree-apiAndrew Earl1-0/+4
2014-03-27AUTHORS: Mention Jussi's contributionsDenis Kenzior1-0/+1
2014-03-27phonesim: Fixup whitespaceDenis Kenzior1-0/+1
2014-03-27phonesim: Add conf file overrideJussi Pakkanen1-2/+5
OFONO_PHONESIM_CONFIG allows the configuration file location to be overriden from the default $CONFIGDIR/phonesim.conf (typically /etc)
2014-03-24gdbus: Add g_dbus_proxy_set_property_arrayClaudio Takahasi2-0/+92
This patch adds a new gdbus utility function to allow setting a property of fixed, and non-fixed values array.
2014-03-20he910: Fix unused variable warningDenis Kenzior1-1/+0
plugins/he910.c: In function ‘qss_query_cb’: plugins/he910.c:174:21: error: unused variable ‘data’ [-Werror=unused-variable]
2014-03-19he910: Switch to permissive parserDenis Kenzior1-1/+1
Seems the CCFC implementation on Telit firmware is not v.250 compliant: ofonod[445]: Aux: > AT+CCFC=0,2\r ofonod[445]: Aux: < \r\n\r\n+CCFC:0,7\r\n\r\nOK\r\n
2014-03-19he910: Fixup SIM detectionDenis Kenzior1-7/+37
In certain cases the #QSS notification is never sent, or sent too early. Make sure that we enable #QSS notification only after the modem has been set as powered. Also make sure that we explicitly query #QSS in case we missed the earlier #QSS notification.
2014-03-19hfp: Use enum from hfp.hDenis Kenzior3-23/+16
2014-03-19hfp: Fixup variable namesDenis Kenzior1-3/+3
2014-03-19hfp: Add CHLD features to drver dataKrzysztof Wilk1-0/+3
2014-03-19handsfree: Fixup variable namesDenis Kenzior1-10/+11
2014-03-19hfp: extend Features to three way calling and CHLDKrzysztof Wilk1-3/+31
2014-03-19include: Declaration function to set CHLD featuresKrzysztof Wilk1-0/+2
2014-03-19hfp: Define hfp_ag_chld_featureKrzysztof Wilk1-0/+11
2014-03-19doc: Extend Features propertyKrzysztof Wilk1-0/+6
2014-03-14AUTHORS: Mention Alfonso's contributionsDenis Kenzior1-0/+1
2014-03-14unit: Fix test to use valid EF_PNNAlfonso Sanchez-Beato1-5/+7
test-simutil was not using EF_PNN files with a valid format.