<div dir="ltr"><div><div><div><div><div>Hi Guys!<br><br>> If you have time, please check if there are other occurrences in OpenBSC<br>> or OsmocomBB where the TMSI is printed as integer.  Thanks!<br><br></div>No problem! :)<br><br>> I think it was a bit too quick. I foresee one problem. Let's assume 
someone<br>> is using TMSIs and now upgrade the sourcecode. All CM Service 
Requests<br>> will fail because the TMSI is not known. What is the 
migration/mitigation plan?<br><br>I absolutely agree with Holger. Maybe we can add some code that will check<br>if database still stores TMSIs in old representation style and convert <span id="result_box" class="" lang="en"><span class="">them to<br></span></span>uint32_t? We can change the libmsc/db.c:db_prepare() for this purpose.<br><br>> tmsi_from_string will not work for this anymore.<br></div><br></div>Yes, I forgot to change the gsm_subscriber.c ... Sorry.<br><br></div>> there is no length check but that doesn't seem to be a big issue right now.<br><br></div>We can just write a function that will do this check instead of using #define.<br><br>> * a DB schema upgrade and store the TMSI as uint32_t<br>
> * Use hex presentation in VTY<br><br>+1<br>
<br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div>С наилучшими пожеланиями,<br></div>Яницкий Вадим.<br></div></div></div>
<br><div class="gmail_quote">2016-03-17 19:43 GMT+06:00 Harald Welte <span dir="ltr"><<a href="mailto:laforge@gnumonks.org" target="_blank">laforge@gnumonks.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Holger,<br>
<span class=""><br>
On Thu, Mar 17, 2016 at 02:29:33PM +0100, Holger Freyther wrote:<br>
> I think it was a bit too quick. I foresee one problem. Let's assume<br>
> someone is using TMSIs and now upgrade the sourcecode. All CM Service<br>
> Requests will fail because the TMSI is not known. What is the<br>
> migration/mitigation plan?<br>
<br>
</span>ugh.  I failed to see that for some strange reason we store the TMSI as<br>
"TEXT" in the database.  That's really odd, especailly as we use %u<br>
formatting when searching for TMSIs in the database.<br>
<span class=""><br>
> * I think the above hunk should be reverted<br>
<br>
</span>done.<br>
<span class=""><br>
> * a DB schema upgrade and store the TMSI as uint32_t<br>
<br>
</span>correct.<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
- Harald Welte <<a href="mailto:laforge@gnumonks.org">laforge@gnumonks.org</a>>           <a href="http://laforge.gnumonks.org/" rel="noreferrer" target="_blank">http://laforge.gnumonks.org/</a><br>
============================================================================<br>
"Privacy in residential applications is a desirable marketing option."<br>
                                                  (ETSI EN 300 175-7 Ch. A6)<br>
</div></div></blockquote></div><br></div>