<p>Richard Mudgett has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/8129">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">manager.c: Fix potential memory leak and corruption.<br><br>ast_str_append_event_header() could potentially leak and corrupt memory if<br>the ast_str needed to expand to add the AMI event header.<br><br>Change-Id: I92f36b855540743b208d76e274152ee2d758176d<br>---<br>M main/manager.c<br>1 file changed, 5 insertions(+), 8 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/29/8129/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/main/manager.c b/main/manager.c<br>index 4e611a0..615573f 100644<br>--- a/main/manager.c<br>+++ b/main/manager.c<br>@@ -9475,19 +9475,16 @@<br> }<br> <br> int ast_str_append_event_header(struct ast_str **fields_string,<br>-                                  const char *header, const char *value)<br>+       const char *header, const char *value)<br> {<br>-   struct ast_str *working_str = *fields_string;<br>-<br>-     if (!working_str) {<br>-          working_str = ast_str_create(128);<br>-           if (!working_str) {<br>+  if (!*fields_string) {<br>+               *fields_string = ast_str_create(128);<br>+                if (!*fields_string) {<br>                        return -1;<br>            }<br>-            *fields_string = working_str;<br>         }<br> <br>- ast_str_append(&working_str, 0,<br>+  ast_str_append(fields_string, 0,<br>              "%s: %s\r\n",<br>               header, value);<br> <br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/8129">change 8129</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/8129"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I92f36b855540743b208d76e274152ee2d758176d </div>
<div style="display:none"> Gerrit-Change-Number: 8129 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>