diff options
| author | Neels Hofmeyr <neels@hofmeyr.de> | 2018-10-01 15:51:18 +0200 |
|---|---|---|
| committer | Neels Hofmeyr <neels@hofmeyr.de> | 2018-10-01 15:58:20 +0200 |
| commit | ea6f519ff84c31acd2689e92adb4dc1688dd9a7a (patch) | |
| tree | cf8b858f036e80361634d7979a160c1f67de7533 /include | |
| parent | 40f35212c71e8926b0dfe7463af959f00c56cd82 (diff) | |
vty logging: fix crash when missing 'logging enable'
We must explicitly check the return value of osmo_log_vty2tgt(), or may run
into a segfault. I wasn't aware of this and introduced numerous such cases in
I36f17c131cc70ce5a1aef62fd9693097de230cd4.
Reproduce: on the VTY, do not issue 'logging enable', invoke 'logging level
force-all LEVEL' first.
Show in logging_vty_test.vty that this situation is now again amended by
telling the user that logging was not enabled.
Related: OS#3611
Change-Id: Id11702d1801d6654ca5e5a51b597a0d802e2e8dd
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions