<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7445">View Change</a></p><p>Patch set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Did you look into why this was done at all?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yes I did.  It went in with the 5982bdcb7c45d286ac7a84ab53fb4a74207d2467 change.  That change is adding new security events to chan_sip.  There is no explanation why struct ast_security_event_successful_auth.using_password was declared this way.  Its usage is patterned after AST_EVENT_IE_EVENT_VERSION.  However, struct ast_security_event_common.version is declared correctly as a uint32_t.  It is main/security_events.c:add_json_object() which has AST_EVENT_IE_EVENT_VERSION and AST_EVENT_IE_USING_PASSWORD executing the same switch case code to read the value out of the struct.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7445">change 7445</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.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.asterisk.org/7445"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ia08bc797613a62f07e5473425f9ccd8d77c80935 </div>
<div style="display:none"> Gerrit-Change-Number: 7445 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 05 Dec 2017 13:18:51 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>