command-nas: fix out-of-bounds read

Limit iteration to number of array members fixing potential
out-of-bounds read.

Coverity CID: 1493471 Out-of-bounds read
Signed-off-by: Daniel Golle <daniel@makrotopia.org>

uqmi: use unmodified upstream JSON files

Extend code generator to handle also identifiers starting with a
numerical character.

Suggested-by: Koen Vandeputte <koen.vandeputte@citymesh.com>
Acked-by: Oskari Lemmelä <oskari@lemmela.net>
Signed-off-by: Daniel Golle <daniel@makrotopia.org>

uim: add --uim-get-sim-state

Provide same output as --get-pin-status for SIM/modem which require
using uim instead of dms command for that.
For now only the first present SIM card with only the first application
on the card is supported (which should cover the vast majority of
cases and allows to remain compatible with --get-pin-status output)

Suggested-by: Arjun AK <arjunak234@gmail.com>
Signed-off-by: Daniel Golle <daniel@makrotopia.org>

uqmi: add more diagnostics commands

Useful diagnostic commands for measuring connection info.
Implemented cell location, carrier aggregation and MIMO chain
info in easily parsable json format.

Compressed package size increased by 9kB.

Signed-off-by: Oskari Lemmela <oskari@lemmela.net>

uqmi: sync data from libqmi project

Sync all json and header files from libqmi project.
Update code to adapt changes in libqmi files.

Signed-off-by: Oskari Lemmela <oskari@lemmela.net>

uqmi: update code generator

Add support for common-refs and double arrays.
Fix error message line and type parsing.

Signed-off-by: Oskari Lemmela <oskari@lemmela.net>

uqmi: add timeout parameter

Working with Quectel EM12 LTE-module, we observe
regular stalls of the QMI interface which cause
a request issued by uqmi to hang forever.

Most reproducibly this happens after the device
has been power-cycled and left untouched for a
while (~ 60s+). Most of the time the very first
QMI request fails, since it is not responded by
the module. This is the strace from such a run
(from --get-pin-status):

 open("/dev/cdc-wdm0", O_RDWR|O_EXCL|O_NOCTTY|O_NONBLOCK|O_LARGEFILE) = 6
 fcntl64(6, F_GETFL)                     = 0x10802 (flags O_RDWR|O_NONBLOCK|O_LARGEFILE)
 fcntl64(6, F_SETFL, O_RDWR|O_NONBLOCK|O_LARGEFILE) = 0
 epoll_ctl(3, EPOLL_CTL_ADD, 6, {EPOLLIN|EPOLLRDHUP|EPOLLET, {u32=268567076, u64=1153486808202346496}}) = 0
 write(6, "\1\17\0\0\0\0\0\1\"\0\4\0\1\1\0\2", 16) = 16
 clock_gettime(CLOCK_MONOTONIC, {tv_sec=95, tv_nsec=583444789}) = 0
 clock_gettime(CLOCK_MONOTONIC, {tv_sec=95, tv_nsec=583770264}) = 0
 epoll_pwait(3,
 [ hang forever ]

After killing the blocked uqmi process, the next
request works as expected.

We don't know whether this is a device FW issue
(we use the latest EM12GPAR01A15M4G) or whether
the device enters some undocumented power-save
mode after idling for some time.

This patch extends uqmi with a timeout option
(-t, --timeout <ms>) which if set terminates a
request after the given amount of msecs. In
our usecase it provides a means of preventing
infinitively stuck QMI requests. Since we
observe the issue only for the very first
request after cold-boot, we use a dummy access
early in qmi.sh, e.g.
  uqmi -d /dev/cdc-wdm0 --get-pin-status -t 3000 >/dev/null 2>&1

This ensures the QMI interface is un-stuck in
case it entered the stall-state observed. The
change is intentionally not included in this
commit, since you don't need it if it works
for you.

Signed-off-by: Zefir Kurtisi <zefir.kurtisi@neratec.com>

nas: add --get-plmn

This command is needed in the qmi proto handler to check if the plmn
is already set to 'auto'.

The reason for this is, that setting the plmn to 'auto' will implicitly
lead to a (delayed) network re-registration, which could further lead
to some timing related issues in the qmi proto handler.

Signed-off-by: Martin Schiller <ms@dev.tdt.de>

uqmi: add explicit check for message type when expecting a response

When the utility sends a request it expects a response type message,
but does not explicitly check for it. When a device stays idle for
some time, it switches into a sleep mode, and notifies the utility with an
identification type message. In some configurations the device only sends
this identification message when triggered by the utility, in this case by
the request message. What the utility gets is two messages at the same time -
an identification and a response. When it tries to decode former it obviously
fails, because it is not what it expected.

Signed-off-by: Tautvydas Belgeras <tautvydas.b@8devices.com>

uqmi_add_command: fixed command argument assignment

The char *arg passed to uqmi_add_command was not assigned to cmd[idx].arg.

Signed-off-by: Carlo Lobrano <c.lobrano@gmail.com>

uqmi: Change returned value to QMI_CMD_REQUEST for 'sync' command.

QMI_CMD_DONE is not working properly with 'sync' command.
We must use QMI_CMD_REQUEST.

Signed-off-by: Nickolay Ledovskikh <nledovskikh@gmail.com>

uqmi: Add sync command to release all cids.

Signed-off-by: Nickolay Ledovskikh <nledovskikh@gmail.com>
Signed-off-by: Felix Fietkau <nbd@nbd.name> [add command line help]

mbim: fix memory corruption issues

Use a common buffer that has enough headroom for the MBIM header.

Signed-off-by: Felix Fietkau <nbd@nbd.name>

nas: fix network scan

Signed-off-by: Matti Laakso <malaakso@elisanet.fi>

wds: add option for specifying profile index

Some modems require the specification of a profile index which
determines the APN and the IP family used for the connection.

Signed-off-by: Matti Laakso <malaakso@elisanet.fi>

wds: make APN optional

Some modem firmwares do not use the APN selected in the Start
Network command. Moreover, it is marked optional in the QMI
specification.

Signed-off-by: Matti Laakso <malaakso@elisanet.fi>

wds: add a separate --set-ip-family option

Change the current ip-family option to set the IP Family Preference
TLV in the Start Network command, and add a separate set-ip-family
option for the Set IP Family command. This is done to match how
the autoconnect and set-autoconnect options work.

Older modems don't support the Set IP Family command, and they
should use the IP Family Preference TLV. On the other hand, newer
modems, e.g., from Huawei, don't respect that TLV, and require the
use of the separate Set IP Family command.

Signed-off-by: Matti Laakso <malaakso@elisanet.fi>

wds: fix set-autoconnect help text

Signed-off-by: Matti Laakso <malaakso@elisanet.fi>

uim: add missing extra data for UIM PIN verify request

Signed-off-by: Felix Fietkau <nbd@nbd.name>

add UIM verify pin commands

Signed-off-by: Felix Fietkau <nbd@nbd.name>

enable uim message functions

Signed-off-by: Felix Fietkau <nbd@nbd.name>

data: update qmi-service-uim.json from libqmi git

Signed-off-by: Felix Fietkau <nbd@nbd.name>

uqmi: fix big endian bugs in MBIM code

All MBIM integer fields are 32bit little endian.  So we need to convert
then all on big endian systems.

Fixes: e69bf24b00d8 ("uqmi: add support for MBIM devices with QMI service")
Signed-off-by: Bjørn Mork <bjorn@mork.no>

dms: add "Set FCC Authentication" request

Many newer modems from Sierra Wireless includes a feature known as
"FCC Authentication".  The basic idea is that the radio is kept in
low power mode until the driver has told the firmware that it is
OK to switch it on.  This is done with a vendor specific QMI DMS
request with no input or output TLVs.

The "Set FCC Authentication" request is required for any modem with
the feature enabled.  Which includes most newer Lenovo branded Sierra
Wireless modems based on Qualcomm chipsets, like for example the
EM7455.

Sample session with an EM7455 in MBIM mode and the FCC Authentication
feature enabled:

 $ umbim -d /dev/cdc-wdm0 -n radio
   hwradiostate: on
   swradiostate: off
 $ uqmi -m -d /dev/cdc-wdm0 --fcc-auth
 $ umbim -d /dev/cdc-wdm0 -t 2 radio
   hwradiostate: on
   swradiostate: on

Signed-off-by: Bjørn Mork <bjorn@mork.no>

uqmi: add support for MBIM devices with QMI service

Many Qualcomm based devices offer a QMI service when running in MBIM
mode. This is useful for a number of requests which have no MBIM
counterpart.

This implementation is very basic, simply wrapping the QMI requests
in an MBIM command. It is up to the user to make sure that the MBIM
command sequence is valid, using a mix of umbim and uqmi requests.

umbim must be used to send "OPEN" before uqmi can issue any MBIM
requests. Example:

1. use umbim to open the session, using the '-n' option:

$ umbim -d /dev/cdc-wdm0 -n caps
  devicetype: 0003 - remote
  cellularclass: 0001
  voiceclass: 0001 - no-voice
  simclass: 0002
  dataclass: 003C
  smscaps: 0003
  controlcaps: 0001
  maxsessions: 0008
  deviceid: 0145820007xxxxx
  firmwareinfo: SWI9X30C_02.08.02.00
  hardwareinfo: EM7455

2. use uqmi to send an MBIM request, using the '-m' option:

$ uqmi -m -d /dev/cdc-wdm0 --get-serving-system
{
        "registration": "registered",
        "plmn_mcc": 242,
        "plmn_mnc": 1,
        "plmn_description": "TELENOR",
        "roaming": false
}

3. use umbim to close the open session, using the '-t X' option:

$ umbim -d /dev/cdc-wdm0 -t 2 caps
  devicetype: 0003 - remote
  cellularclass: 0001
  voiceclass: 0001 - no-voice
  simclass: 0002
  dataclass: 003C
  smscaps: 0003
  controlcaps: 0001
  maxsessions: 0008
  deviceid: 0145820007xxxxx
  firmwareinfo: SWI9X30C_02.08.02.00
  hardwareinfo: EM7455

Signed-off-by: Bjørn Mork <bjorn@mork.no>
Signed-off-by: Felix Fietkau <nbd@nbd.name> [cleanup, portability fixes]

build: Support for out of source builds added

The scripts gen-{header,code,error-list}.pl were called with
source-local paths what disallowed out of source builds.

Updated CMakeLists.txt to call the scripts with ${CMAKE_SOURCE_DIR}
prefix.

Signed-off-by: Dmitri Bachtin <dbachtin@init-ka.de>

build: added _library suffix to lib variables

It is a convention under CMake to suffix variables in the FIND_LIBRARY
calls with _LIBRARY. Added _library suffix to json, blobmsg_json and
ubox variables.

uqmi: search for ubox, blobmsg_json and json include paths

CMake will search for include paths via FIND_PATH() of the following
libraries: libubox, blobmsg_json and json. This allows to build uqmi
when the dependencies are in non-standard locations by specifying
respective _include_dir variables.

wds: cast pdp_type enum to int to avoid clang considering the range check tautological

enum overflow behavior is undefined

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

wds: make ipv6 address endian swap more portable

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

uqmi: Added CMake option BUILD_STATIC for a static uqmi build

Signed-off-by: Dmitri Bachtin <dbachtin@init-ka.de>

uqmi: Add get-current-settings command-line switch

Adding --get-current-settings switch which can be used to query
ip-settings among some other useful data obtained from remote end when
connected.

This is mainly useful with modems which do not provide a dhcp server for
nameserver or ip-information (especially in ipv6 networks).

Signed-off-by: Antti Seppälä <a.seppala@gmail.com>
Tested-by: Matti Laakso <malaakso@elisanet.fi>

uqmi: Add IP family selection command-line switch

This patch adds support for (optionally) specifying ip family via a
command- line switch. The switch sends respective "Set IP Family" WDS
message to qmi-device before actually connecting.

Using this switch allows connecting to ipv6 enabled networks or networks
with dual-stack support with the appropriate hardware (dongle and FW
with ipv6 support) and configuration (AT+CGDCONT reporting ipv6 or
ipv4v6 capability).

Help text:
  --ip-family <family>:    Set ip-family for the connection (ipv4, ipv6, unspecified)

Usage example for ipv6:
  uqmi -d /dev/cdc-wdm0 --set-client-id wds,<cid> --start-network <apn> --ip-family ipv6

Dual-stack usage example:
  uqmi -d /dev/cdc-wdm0 --get-client-id wds
  uqmi -d /dev/cdc-wdm0 --set-client-id wds,<cid-1> --start-network <apn> --ip-family ipv4
  uqmi -d /dev/cdc-wdm0 --get-client-id wds
  uqmi -d /dev/cdc-wdm0 --set-client-id wds,<cid-2> --start-network <apn> --ip-family ipv6

Signed-off-by: Antti Seppälä <a.seppala@gmail.com>
Tested-by: Matti Laakso <malaakso@elisanet.fi>

add missing license headers (LGPL v2)

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

Add command to get IMEI from the device

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

Add command to specify preferred PLMN

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Add command for listing device capabilities

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Add missing option for getting tdma signal strength

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Fix encoding phone number on sending sms

Return proper length of encoded phone number with odd count of digits,
without this patch number is encoded in memory but length returned
by function is not incremented and sms cannot be send.

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Making smsc number optional

If smsc number is not provided before sending sms then
operator's default stored on simcard will be used.

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Fix improper length of sms message stored in User Data Length field

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Fix logical expression which is always true

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Add mandatory field to Initiate Network Register

According to json data Action field in Initiate Network Register
is mandatory. I get "Missing argument" from qmi without it.

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Fix improper characters after reading Escape code for switching extension table in SMS

Escape code 0x1b is a prefix indicating that next character should be
represented from extension table, but it only applies to one character,
so switching it off is needed to read further characters correctly.

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Added commands to change PIN code

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Added shared struct to allow the same option names for different functions

For example we can use one option --new-pin with commands unblock-pin
and with change-pin, without that we need to invent new name
for every new command.

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Added option for enabling and disabling protection of simcard by PIN

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Add support for 16-bit reference number in concatenated SMS.

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Fix character "@" added at end of received messages

If 7 bit message encoded on 8 bit cells has one last bit occupying
whole byte it need to be padded with 7 zero bits. To not mistake
these last 7 bits with a character with code 0 (@) we need to check
length from User Data Length field and not rely on actual length
of received data. An example can be message "abcdefg" encoded
as "61 f1 98 5c 36 9f 01", it is decoded as "abcdefg@".

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Added support for 8 bit and 16 bit encoded SMS messages

Message is presented as string of hexadecimal pairs in JSON output.

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Separation of decoding User Data Header from decoding 7 bit message

It is preparation for supporting 8 bit and 16 bit encoding. Moving out
this code from decode_7bit_field() allows to reuse it in caller
function where other than 7 bit decoding will take place.

Signed-off-by: SÅ‚awomir Demeszko <s.demeszko@wireless-instruments.com>

Add --delete-message for deleting SMS messages

Signed-off-by: Sławomir Demeszko <s.demeszko@wireless-instruments.com>

SEGFAULT on reading Unicode sms messages

Added complementary blobmsg_close_table() before returning from function
on error.

add wda commands for setting/getting wireless data mode

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

suppress enum conversion warnings

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

fix indentation

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

Fix improper part number in multipart SMS text messages

Received multipart SMS messages has part number always one too high.

Add command --get-msisdn for getting phone number

Added command "--get-iccid" to read the SIM serial number.

Added commands to unblock blocked PINs.

Added commands "--unblock-pin1" and "--unblock-pin2" and
parameters "--puk" and  "--new-pin".

add a command for setting the interface data format

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

fix aliasing error for parsing message float data

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

add a command for changing the autoconnect setting

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

silence a compiler warning about adding an int to a string

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

make argument to get_pin_status int instead of enum to prevent invalid optimization

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

move a variable declaration to the right place

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

fix tautological check

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

sync with libqmi-glib

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

Fix printing of service versions

Signed-off-by: Matti Laakso <malaakso@elisanet.fi>

Add --stop-network

Signed-off-by: Matti Laakso <malaakso@elisanet.fi>

Fix autoconnect

Signed-off-by: Matti Laakso <malaakso@elisanet.fi>

Minor fixes to command line help

Signed-off-by: Matti Laakso <malaakso@elisanet.fi>

add --set-device-operating-mode

add --reset-dms

add autoconnect option for --start-network

make size related int variables unsigned

unify multiple set_system_selection commands into one message, add --set-network-roaming

make uqmi_add_error return QMI_CMD_EXIT

fix releasing client ids after explicit set

add wds reset command

fix command type for get_packet_service_status

implement --get-data-status

rename send to _send to avoid conflict with system headers

codegen: work around firmware bug, only accept the first found tlv

add --network-scan

fix generated code for allocating arrays

add an option for printing output as a single line

allow commands to return individual json elements instead of just json objects

format output as json

nas: add --set-network-preference

keep the client id if user-specified

disable unused libraries

fix helptext for verify pin

fix 7bit character conversion

add an option for releasing client ids

fix 7-bit encoding

fix charset decoding of 7-bit gsm data

get rid of unused variables

add support for sending sms

add detection for the class field in the TP-DCS octet