* [PATCH] test: Fix number of arguments check in change-pin
@ 2018-11-27 9:52 Nandini Rebello
2018-11-27 14:45 ` Denis Kenzior
0 siblings, 1 reply; 2+ messages in thread
From: Nandini Rebello @ 2018-11-27 9:52 UTC (permalink / raw
To: ofono
[-- Attachment #1: Type: text/plain, Size: 580 bytes --]
Fix the number of arguments checked in second case of change-pin.
---
test/change-pin | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/change-pin b/test/change-pin
index 301c6ce..b8b1ad7 100755
--- a/test/change-pin
+++ b/test/change-pin
@@ -10,7 +10,7 @@ if len(sys.argv) == 5:
pin_type = sys.argv[2]
old_pin = sys.argv[3]
new_pin = sys.argv[4]
-elif len(sys.argv) == 3:
+elif len(sys.argv) == 4:
manager = dbus.Interface(bus.get_object('org.ofono', '/'),
'org.ofono.Manager')
modems = manager.GetModems()
--
2.7.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] test: Fix number of arguments check in change-pin
2018-11-27 9:52 [PATCH] test: Fix number of arguments check in change-pin Nandini Rebello
@ 2018-11-27 14:45 ` Denis Kenzior
0 siblings, 0 replies; 2+ messages in thread
From: Denis Kenzior @ 2018-11-27 14:45 UTC (permalink / raw
To: ofono
[-- Attachment #1: Type: text/plain, Size: 258 bytes --]
Hi Nandini,
On 11/27/2018 03:52 AM, Nandini Rebello wrote:
> Fix the number of arguments checked in second case of change-pin.
> ---
> test/change-pin | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied, thanks.
Regards,
-Denis
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-11-27 14:45 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-11-27 9:52 [PATCH] test: Fix number of arguments check in change-pin Nandini Rebello
2018-11-27 14:45 ` Denis Kenzior
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.