Change in pysim[master]: cards: return SW in method update_plmnsel() of class Card

[dexter]

dexter has uploaded this change for review. ( https://gerrit.osmocom.org/9986


Change subject: cards: return SW in method update_plmnsel() of class Card
......................................................................

cards: return SW in method update_plmnsel() of class Card

The method update_plmnsel() does not return the status word yet, which
causes pysim-prog.py to print an error message that does not influence
the functionality but does not look nice.

- preserve the status word that is returned with update_binary() and
  return it properly like the other methods do.

Change-Id: I54e8e165f87365e8162b36d24efc8f0db62b66da
Related: SYS#4245
---
M pySim/cards.py
1 file changed, 2 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/86/9986/1

diff --git a/pySim/cards.py b/pySim/cards.py
index 9be04ff..a72a52f 100644
--- a/pySim/cards.py
+++ b/pySim/cards.py
@@ -87,7 +87,8 @@
 	        data = self._scc.read_binary(EF['PLMNsel'], length=None, offset=0)
                 size = len(data[0])/2
 		hplmn = enc_plmn(mcc, mnc)
-		self._scc.update_binary(EF['PLMNsel'], hplmn + 'ff' * (size-3))
+		data, sw = self._scc.update_binary(EF['PLMNsel'], hplmn + 'ff' * (size-3))
+		return sw
 
 	def update_smsp(self, smsp):
 		data, sw = self._scc.update_record(EF['SMSP'], 1, rpad(smsp, 84))

-- 
To view, visit https://gerrit.osmocom.org/9986
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I54e8e165f87365e8162b36d24efc8f0db62b66da
Gerrit-Change-Number: 9986
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180713/66cbf2c1/attachment.htm>