<p>Harald Welte <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/9587">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Harald Welte: Looks good to me, approved; Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">pysim-prog: also allow raw hex adm pins besides numeric ascii<br><br>At the momemnt pysim takes the supplied ADM pin number and interprets<br>it as ascii string. This ascii string (max 8 digitis) is then padded<br>with 0xff bytes if necessary and sent to the card.<br><br>This limits the range of possible ADM keys and it is not possible<br>to deal with situataions where the ADM pin consists of arbitrary<br>bytes. At the momemnt pysim-prog forbis anything that is longer<br>than 8 digits. Lets also check if there are 16 digits and if yes<br>interpret them as raw bytes.<br><br>- when the adm pin is 16 digits long, interpret the string as raw<br>  bytes (hex).<br><br>Change-Id: If0ac0d328c64b57bc4d45d985a4a516930053344<br>Related: SYS#4245<br>---<br>M pySim-prog.py<br>1 file changed, 7 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/pySim-prog.py b/pySim-prog.py</span><br><span>index 0c9f749..d70eac9 100755</span><br><span>--- a/pySim-prog.py</span><br><span>+++ b/pySim-prog.py</span><br><span>@@ -361,10 +361,13 @@</span><br><span>            opc = ''.join(['%02x' % random.randrange(0,256) for i in range(16)])</span><br><span> </span><br><span>     if opts.pin_adm is not None:</span><br><span style="color: hsl(0, 100%, 40%);">-            if len(opts.pin_adm) > 8:</span><br><span style="color: hsl(0, 100%, 40%);">-                    raise ValueError("PIN-ADM needs to be <=8 digits")</span><br><span style="color: hsl(0, 100%, 40%);">-         pin_adm = ''.join(['%02x'%(ord(x)) for x in opts.pin_adm])</span><br><span style="color: hsl(0, 100%, 40%);">-              pin_adm = rpad(pin_adm, 16)</span><br><span style="color: hsl(120, 100%, 40%);">+           if len(opts.pin_adm) <= 8:</span><br><span style="color: hsl(120, 100%, 40%);">+                 pin_adm = ''.join(['%02x'%(ord(x)) for x in opts.pin_adm])</span><br><span style="color: hsl(120, 100%, 40%);">+                    pin_adm = rpad(pin_adm, 16)</span><br><span style="color: hsl(120, 100%, 40%);">+           elif len(opts.pin_adm) == 16:</span><br><span style="color: hsl(120, 100%, 40%);">+                 pin_adm = opts.pin_adm</span><br><span style="color: hsl(120, 100%, 40%);">+                else:</span><br><span style="color: hsl(120, 100%, 40%);">+                 raise ValueError("PIN-ADM needs to be <=8 digits (ascii) or exactly 16 digits (raw hex)")</span><br><span>       else:</span><br><span>                pin_adm = None</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/9587">change 9587</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/9587"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: pysim </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: If0ac0d328c64b57bc4d45d985a4a516930053344 </div>
<div style="display:none"> Gerrit-Change-Number: 9587 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>