From 547cef5afbfbcbf9fe78705c7b5661059b706346 Mon Sep 17 00:00:00 2001 From: Sertonix Date: Mon, 22 Apr 2024 15:47:10 +0200 Subject: [PATCH] network: fix missing break in switch statement This can cause the first character of the string to be read as an iface state. Fixes https://codeberg.org/dnkl/yambar/issues/377 --- CHANGELOG.md | 6 ++++++ modules/network.c | 1 + 2 files changed, 7 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 5221a37..3152882 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -18,6 +18,12 @@ ### Deprecated ### Removed ### Fixed + +* network: fix missing break in switch statement([#377][377]) + +[377]: https://codeberg.org/dnkl/yambar/issues/377 + + ### Security ### Contributors diff --git a/modules/network.c b/modules/network.c index 4bb0fb2..adb8f68 100644 --- a/modules/network.c +++ b/modules/network.c @@ -595,6 +595,7 @@ handle_link(struct module *mod, uint16_t type, const struct ifinfomsg *msg, size iface->name = strdup((const char *)RTA_DATA(attr)); LOG_DBG("%s: index=%d", iface->name, iface->index); mtx_unlock(&mod->lock); + break; case IFLA_OPERSTATE: { uint8_t operstate = *(const uint8_t *)RTA_DATA(attr); if (iface->state == operstate)