modemmanager: fix value call with modemmanager_get_field function

The correct values are prefixed with 'modem.generic'. This is missing
for the value 'state' and 'state-failed-reason.

While we're at it, let's move the readout of 'state-failed-reason' to the
failed case, because that's the only place it's needed.

Signed-off-by: Florian Eckert <fe@dev.tdt.de>

diff --git a/net/modemmanager/files/lib/netifd/proto/modemmanager.sh b/net/modemmanager/files/lib/netifd/proto/modemmanager.sh
index 562f2e3c378a84c760068eba22fa2f90548188ee..9da65935cf21d038b422972fda9c2001351a5984 100644 (file)
--- a/net/modemmanager/files/lib/netifd/proto/modemmanager.sh
+++ b/net/modemmanager/files/lib/netifd/proto/modemmanager.sh
@@ -315,12 +315,11 @@ modemmanager_check_state() {
 
        local state reason
 
-       state="$(modemmanager_get_field "${modemstatus}" "state")"
-       state="${state%% *}"
-       reason="$(modemmanager_get_field "${modemstatus}" "state-failed-reason")"
+       state="$(modemmanager_get_field "${modemstatus}" "modem.generic.state")"
 
        case "$state" in
                "failed")
+                       reason="$(modemmanager_get_field "${modemstatus}" "modem.generic.state-failed-reason")"
                        case "$reason" in
                                "sim-missing")
                                        echo "SIM missing"