Update Ofono to 1.33 - #29
Open
piggz wants to merge 316 commits into
Open
Conversation
We pass in the maximum size of the buffer to the read system call. On the astronomically unlikely chance that we indeed read the full buffer full of data, the subsequent assignment will overflow it. Fix this by passing sizeof(buf) - 1 to the read system call instead.
In case strlen(ICCID) > 20, we simply return without freeing the ICCID value first.
if (ctx->message_proxy) always resolves to TRUE
The default context created when provisioning fails is simply a context with an empty APN
ctx->message_center and ctx->message_proxy are arrays and thus always evaluate as true
The code is not really copying strings but 2 or 3 character sof the MNC. Make this clearer.
When a context gets activated we read its settings, but if the context gets deactivated during this time we should not indicate that the context is activated when the settings are received.
The SIMCom SIM7100 and SIM7600 modem use the same vendor and product id but require different drivers (sim7100 vs gobi).
Added two new values for TechnologyPreference property as follows: "umts,gsm" Dual mode operation with UMTS and GSM radio access whith preference for UMTS. "lte,umts" Dual mode operation with LTE and UMTS radio access with preference for LTE.
Handled two new dual mode technology preferences "umts,gsm" and "lte,umts".
Handled dual mode technology preference "umts,gsm" and "lte,umts" for xmm7modem.
handled enabling of stk for xmm7modem vendor in atmodem driver.
Add support for gprs contexts with username, password, and specific authentication type.
Selection capability for authentication method via a command line argument has been added
Some modems such as Quectel EC200T do not honor the default value for the Async-Control-Character-Map (ACCM) configuration option defined in RFC 1548 6.2 as 0xffffffff. This patch suggests to use RX ACCM = 0 for Ofono by default as pppd does for instance. This will reduce PPP data overhead as well.
Support for the Quectel EC200 USB modem series has been added. The model identification AT command has been added as the first step in the communication with a Quectel USB modem.
Implement network monitoring driver for gemalto modems that are able to provide serving cell information and basic measurements using AT+CQS and AT^SMONI commands.
Enable netmon functionality for Gemalto ELS81x modems.
Gemalto modem reports raw measurements in dBm. Reported values may include negative numbers. Meanwhile oFono follows ETSI TS 27.007, so negative numbers do not really exist at the API level. Modify gemalto netmon driver to report measurements according to 27.007. For this purpose re-scale from what Gemalto firmware reports into something that 27.007 recommends.
Add helper to validate if TLV value is a valid UTF8 string. Note that both null-terminated and non null-terminated UTF8 strings are considered valid.
Make sure that IMS private identity is a valid UTF8 string before setting sim->impi field. Otherwise ofono may crash on dbus assert when SIM properties are reported via org.ofono.SimManager interface.
Handle USSD QMI indication messages. Add support for UCS2 USS Data coding scheme. Check for User Action TLV type.
g_utf8_validate_len() is support after glib 2.60
Added one new value for TechnologyPreference property
as follows:
"lte,gsm" Dual mode operation with LTE
and GSM radio access whith preference
for LTE.
Allow setting of "lte,gsm" mode, for modems that doesn't support ANY mode.
Handled dual mode technology preference "lte,gsm" for Quectel BG96. Quectel BG96 doesn't support ANY mode.
Originally the enum was only meant to signify a single radio access mode preference, but over time its meaning was overloaded to be a bitfield with multiple preferences. Switch away from using an enum value to using an unsigned int to make it clearer that the value is a bitfield.
Add support for Gemalto specific radio settings.
Instantiate Gemalto radio-settings atom in post_sim.
Gemalto modem does not allow to enable Supplementary Service during post_sim stage. So move ussd atom to post_online stage.
Enum ofono_radio_access_mode has been replaced by unsigned int. This change allows to move handling of all the modes into 'switch' in the function gemalto_set_rat_mode.
lduboeuf
pushed a commit
to lduboeuf/ofono-mer
that referenced
this pull request
Nov 21, 2022
Allow configuration of set_udub ril op through ril_subscription.conf
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Builds upon the 1.32 PR and cherry-picks commits to 1.33 with the following changes
Skipped:
https://git.kernel.org/pub/scm/network/ofono/ofono.git/commit/?h=1.34&id=8b4e79b443ee20d287bb68ae3a02c1819fd1b2d5 (jolla changed the code substantially here in dffc04d, JB#54048)
https://git.kernel.org/pub/scm/network/ofono/ofono.git/commit/?h=1.34&id=37745c22019e382b1c1d8e511183c43c38b050ce (already present)
https://git.kernel.org/pub/scm/network/ofono/ofono.git/commit/?h=1.34&id=f10b870c25d40362dd98b22f422fbbdde3ad8059 (already present)
https://git.kernel.org/pub/scm/network/ofono/ofono.git/commit/?h=1.34&id=121448e3785daf68b0f3d4e12967da78fb07388c (already present)