modemmanager: add missing interface argument to function modemmanager_check_state
authorFlorian Eckert <fe@dev.tdt.de>
Thu, 26 Sep 2024 08:16:13 +0000 (10:16 +0200)
committerRobert Marko <robimarko@gmail.com>
Thu, 17 Oct 2024 16:29:16 +0000 (18:29 +0200)
The variable interface is used in the sub-function 'modemmanager_check_state'.
However, this is not an argument of the function and so the global value
is used. Addding the variable as an function call argument fixes this.

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
net/modemmanager/files/lib/netifd/proto/modemmanager.sh

index a0bc232aad300cae9a2378c89665f7d348acf094..0827c27136d0920a78c66b66a9af35c5ee763d0d 100644 (file)
@@ -355,8 +355,9 @@ modemmanager_check_state_locked() {
 
 modemmanager_check_state() {
        local device="$1"
-       local modemstatus="$2"
-       local pincode="$3"
+       local interface="$2"
+       local modemstatus="$3"
+       local pincode="$4"
 
        local state
 
@@ -486,7 +487,7 @@ proto_modemmanager_setup() {
        }
        echo "modem available at ${modempath}"
 
-       modemmanager_check_state "$device" "${modemstatus}" "$pincode"
+       modemmanager_check_state "$device" "$interface" "${modemstatus}" "$pincode"
        [ "$?" -ne "0" ] && return 1
 
        # always cleanup before attempting a new connection, just in case