<p><a href="https://gerrit.osmocom.org/13366">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13366/3/pySim/cards.py">File pySim/cards.py:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13366/3/pySim/cards.py@121">Patch Set #3, Line 121:</a> <code style="font-family:monospace,monospace">        def update_ad(self, mnc):</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">so this is part of the generic "card" class...</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13366/3/pySim/cards.py@603">Patch Set #3, Line 603:</a> <code style="font-family:monospace,monospace">                # EF.AD</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">but it's used only from the sysmoUSIM-SJS1 specicic code.  However, the other cards likely have exactly the same problem.  I hope there's nothing card-model specific about writing EF.AD (a standard file) and hence I would assume we can simply make this common to all card typeS?  If there are cards where writing the file is not allowed, then we have to at least copy this block to all cards that do support it.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13366">change 13366</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/13366"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: I827092b2c7f7952f54b2d9f8dbda419a0dbfaf65 </div>
<div style="display:none"> Gerrit-Change-Number: 13366 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 27 Mar 2019 07:27:13 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>