<p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/15948">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/15948/1/src/libmsc/mncc.c">File src/libmsc/mncc.c:</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/osmo-msc/+/15948/1/src/libmsc/mncc.c@266">Patch Set #1, Line 266:</a> <code style="font-family:monospace,monospace">          * char sdp[] starts with a '\0'. */</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I'd better don't rely on msgb being zero-filled and mark the related byte = 0.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">but msgb_alloc_c() has:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">        /* Manually zero-initialize allocated memory */<br>        memset(msg, 0x00, sizeof(*msg) + size);</pre><p style="white-space: pre-wrap; word-wrap: break-word;">So why, are you aiming at future alternative msgb implementations?</p><p style="white-space: pre-wrap; word-wrap: break-word;">To write a '\0' to a msgb's unused data part, we would have to manually check that we're not writing past the allocated mem. I'd rather not dup that validation out of msgb.c...</p><p style="white-space: pre-wrap; word-wrap: break-word;">Ah ok, I could msgb_put_u8() and then remove that again.</p><p style="white-space: pre-wrap; word-wrap: break-word;">But seems unnecessary?</p><p style="white-space: pre-wrap; word-wrap: break-word;">Plus I'd rather not modify the msgb we got from incoming data, as a sane premise.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-msc/+/15948">change 15948</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/c/osmo-msc/+/15948"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ie16f0804c4d99760cd4a0c544d0889b6313eebb7 </div>
<div style="display:none"> Gerrit-Change-Number: 15948 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 04 Nov 2019 15:05:51 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>