Change in libosmocore[master]: gsm_utils: fix use buf_len instead of sizeof in osmo_dump_gsmtime_buf

dexter gerrit-no-reply at lists.osmocom.org
Mon Apr 8 10:16:25 UTC 2019


Hello Pau Espin Pedrol, Jenkins Builder, 

I'd like you to reexamine a change. Please visit

    https://gerrit.osmocom.org/13547

to look at the new patch set (#2).

Change subject: gsm_utils: fix use buf_len instead of sizeof in osmo_dump_gsmtime_buf
......................................................................

gsm_utils: fix use buf_len instead of sizeof in osmo_dump_gsmtime_buf

The function osmo_dump_gsmtime_buf gets a pointer *buf and a parameter
buf_len. The pointer *buf is a string buffer and the function places an
\0 at the end of the buffer before it exists. However it uses
sizeof(buf) as part of the index calculation, which is incorrect. Lets
correct this by using buf_len instead.

Change-Id: Id24263aa7c9a53544f1639b6ceb09ce5615d5114
---
M src/gsm/gsm_utils.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/47/13547/2
-- 
To view, visit https://gerrit.osmocom.org/13547
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id24263aa7c9a53544f1639b6ceb09ce5615d5114
Gerrit-Change-Number: 13547
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190408/e6cfb2f9/attachment.html>


More information about the gerrit-log mailing list