openggsn.git branch master updated. 0.92-29-gf41f586

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/osmocom-commitlog@lists.osmocom.org/.

gitosis at osmocom.org gitosis at osmocom.org
Fri Oct 14 11:40:05 UTC 2016


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The OpenGGSN project".

The branch, master has been updated
       via  f41f5866cebbf88b6ae09ec8bd00c308f4d9f315 (commit)
       via  f7611c3ceea4afffeded41e287a3200a578e8a5b (commit)
      from  38929c9131b606d074d438a6476c5b032badb46d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://cgit.osmocom.org/openggsn/commit/?id=f41f5866cebbf88b6ae09ec8bd00c308f4d9f315

commit f41f5866cebbf88b6ae09ec8bd00c308f4d9f315
Author: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Date:   Mon Sep 19 03:35:53 2016 +0200

    gsn_restart file: wrap umask change tightly around file creation
    
    An fopen("w") error used to omit the umask() call to reinstate the previous
    umask. Also an fopen("r") at the top will never create a file and hence does
    not need a umask set.
    
    Instead, wrap the umask change and change-back tightly around the single
    fopen("w") call.
    
    Change-Id: If7d948e2f2ba47ecebba5614f18235a53b273d14

http://cgit.osmocom.org/openggsn/commit/?id=f7611c3ceea4afffeded41e287a3200a578e8a5b

commit f7611c3ceea4afffeded41e287a3200a578e8a5b
Author: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Date:   Thu Aug 18 03:53:09 2016 +0200

    fix gsn_restart file buffer overflow and missing path sep
    
    Fix errors during gsn_restart file path composition:
    
    - possible buffer overflow because the wrong remaining length was fed to
      strncat().
    - missing path separator: put restart file in dir/gsn_restart instead of
      ../dirgsn_restart.
    
    This assumes that the path separator is '/'.
    
    Use talloc_asprintf() to fix all filename length problems and shorten the code.
    
    In order to free the allocated path, add a free_filename label, and jump there
    instead of returning from the fopen("w") failure branch. Also don't return from
    "fclose failed" branch in order to free the path, remove the if {} braces.
    
    Change-Id: Idf0a64ff45720aa818f2f9de1e8ba2fe2c82631b

-----------------------------------------------------------------------

Summary of changes:
 gtp/gtp.c | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)


hooks/post-receive
-- 
The OpenGGSN project



More information about the osmocom-commitlog mailing list