<p>fixeria <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-trx/+/25935">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, but someone else must approve
  osmith: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">trx_rate_ctr: use thread safe strerror() in device_sig_cb()<br><br>Change-Id: Ibd52af22bbe99652f402ada87410de167a124b73<br>---<br>M CommonLibs/trx_rate_ctr.cpp<br>1 file changed, 5 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/CommonLibs/trx_rate_ctr.cpp b/CommonLibs/trx_rate_ctr.cpp</span><br><span>index ba4fcc0..a7467c1 100644</span><br><span>--- a/CommonLibs/trx_rate_ctr.cpp</span><br><span>+++ b/CommonLibs/trx_rate_ctr.cpp</span><br><span>@@ -214,6 +214,7 @@</span><br><span>     struct timespec next_sched = {.tv_sec = 0, .tv_nsec = 20*1000*1000};</span><br><span>         /* no automatic re-trigger */</span><br><span>        struct timespec intv_sched = {.tv_sec = 0, .tv_nsec = 0};</span><br><span style="color: hsl(120, 100%, 40%);">+     char err_buf[256];</span><br><span> </span><br><span>       switch (signal) {</span><br><span>    case S_DEVICE_COUNTER_CHANGE:</span><br><span>@@ -222,7 +223,8 @@</span><br><span>          dev_rate_ctr_mutex.lock();</span><br><span>           dev_ctrs_pending[dev_ctr->chan] = *dev_ctr;</span><br><span>               if (osmo_timerfd_schedule(&dev_rate_ctr_timerfd, &next_sched, &intv_sched) < 0) {</span><br><span style="color: hsl(0, 100%, 40%);">-                        LOGC(DCTR, ERROR) << "Failed to schedule timerfd: " << errno << " = "<< strerror(errno);</span><br><span style="color: hsl(120, 100%, 40%);">+                  LOGC(DCTR, ERROR) << "Failed to schedule timerfd: " << errno</span><br><span style="color: hsl(120, 100%, 40%);">+                                      << " = "<< strerror_r(errno, err_buf, sizeof(err_buf));</span><br><span>              }</span><br><span>            dev_rate_ctr_mutex.unlock();</span><br><span>                 break;</span><br><span>@@ -232,7 +234,8 @@</span><br><span>                 trx_rate_ctr_mutex.lock();</span><br><span>           trx_ctrs_pending[trx_ctr->chan] = *trx_ctr;</span><br><span>               if (osmo_timerfd_schedule(&trx_rate_ctr_timerfd, &next_sched, &intv_sched) < 0) {</span><br><span style="color: hsl(0, 100%, 40%);">-                        LOGC(DCTR, ERROR) << "Failed to schedule timerfd: " << errno << " = "<< strerror(errno);</span><br><span style="color: hsl(120, 100%, 40%);">+                  LOGC(DCTR, ERROR) << "Failed to schedule timerfd: " << errno</span><br><span style="color: hsl(120, 100%, 40%);">+                                      << " = "<< strerror_r(errno, err_buf, sizeof(err_buf));</span><br><span>              }</span><br><span>            trx_rate_ctr_mutex.unlock();</span><br><span>                 break;</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-trx/+/25935">change 25935</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-trx/+/25935"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-trx </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ibd52af22bbe99652f402ada87410de167a124b73 </div>
<div style="display:none"> Gerrit-Change-Number: 25935 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>