<div dir="ltr">Hi all,<div><br></div><div>I've just found a regression in the syslog functionality in v4 std module. Not sure when/where it emerged.</div><div><br></div><div>The log(...) method and the syslog(...) methods where updated to use workspace. But the implementation is slightly different and the result is that the syslog method writes an empty string to syslog.</div><div><br></div><div><div><span class="" style="white-space:pre">...</span></div><div><span class="" style="white-space:pre">     </span>CHECK_OBJ_NOTNULL(ctx, VRT_CTX_MAGIC);</div><div><span class="" style="white-space:pre">     </span>u = WS_Reserve(ctx->ws, 0);</div><div><span class="" style="white-space:pre">     </span>p = ctx->ws->f;</div><div><span class="" style="white-space:pre">      </span>va_start(ap, fmt);</div><div><span class="" style="white-space:pre"> </span>p = VRT_StringList(p, u, fmt, ap);</div><div><span class="" style="white-space:pre"> </span>va_end(ap);</div><div><span class="" style="white-space:pre">        </span>if (p != NULL)</div><div><span class="" style="white-space:pre">     </span>  syslog((int)fac, "%s", p);</div><div><span class="" style="white-space:pre">    </span>WS_Release(ctx->ws, 0);</div></div><div>...</div><div><br></div><div>I think it should be:</div><div><br></div><div><div><span class="" style="white-space:pre">...</span></div><div><b><span class="" style="white-space:pre"> </span>txt t;</b></div><div><span class="" style="white-space:pre">   </span>CHECK_OBJ_NOTNULL(ctx, VRT_CTX_MAGIC);<br></div><div><span class="" style="white-space:pre">   </span>u = WS_Reserve(ctx->ws, 0);</div><div><b><span class="" style="white-space:pre">    </span>t.b = ctx->ws->f;</b></div><div><span class="" style="white-space:pre">  </span>va_start(ap, fmt);<br></div><div><b><span class="" style="white-space:pre">      </span>t.e = VRT_StringList(t.b, u, fmt, ap);</b></div><div><span class="" style="white-space:pre">   </span>va_end(ap);</div><div><b><span class="" style="white-space:pre">       </span>if (t.e != NULL) {</b></div><div><b><span class="" style="white-space:pre">      </span>  assert(t.e > t.b);</b></div><div><b><span class="" style="white-space:pre">        </span>  t.e--;</b></div><div><b><span class="" style="white-space:pre">       </span>  syslog((int)fac, "%s", t.b);</b><br></div><div><b><span class="" style="white-space:pre">     </span>}</b></div><div><span class="" style="white-space:pre">        </span>WS_Release(ctx->ws, 0);<br></div><div>...</div></div><div><br></div><div>Thoughts?</div><div><br></div><div>-phil</div><div><br></div><div><br></div></div>