diff options
| author | Stefano Brivio <sbrivio@redhat.com> | 2025-04-30 16:48:34 +0200 |
|---|---|---|
| committer | Stefano Brivio <sbrivio@redhat.com> | 2025-04-30 16:58:58 +0200 |
| commit | ea0a1240df671de221f469327899564ed74b5edd (patch) | |
| tree | 989a5e090202ad651d2e5e9e361308b2054e5db8 | |
| parent | aa1cc8922867b8f7c17742f8da3b9fcc6291bbeb (diff) | |
| download | passt-ea0a1240df671de221f469327899564ed74b5edd.tar passt-ea0a1240df671de221f469327899564ed74b5edd.tar.gz passt-ea0a1240df671de221f469327899564ed74b5edd.tar.bz2 passt-ea0a1240df671de221f469327899564ed74b5edd.tar.lz passt-ea0a1240df671de221f469327899564ed74b5edd.tar.xz passt-ea0a1240df671de221f469327899564ed74b5edd.tar.zst passt-ea0a1240df671de221f469327899564ed74b5edd.zip | |
passt-repair: Hide bogus gcc warning from -Og
When building with gcc 13 and -Og, we get:
passt-repair.c: In function ‘main’:
passt-repair.c:161:23: warning: ‘ev’ may be used uninitialized [-Wmaybe-uninitialized]
161 | if (ev->len > NAME_MAX + 1 || ev->name[ev->len - 1] != '\0') {
| ~~^~~~~
but that can't actually happen, because we only exit the preceding
while loop if 'found' is true, and that only happens, in turn, as we
assign 'ev'.
Get rid of the warning by (redundantly) initialising ev to NULL.
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
| -rw-r--r-- | passt-repair.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/passt-repair.c b/passt-repair.c index 256a8c9..ff1c44f 100644 --- a/passt-repair.c +++ b/passt-repair.c @@ -113,7 +113,7 @@ int main(int argc, char **argv) if ((sb.st_mode & S_IFMT) == S_IFDIR) { char buf[sizeof(struct inotify_event) + NAME_MAX + 1] __attribute__ ((aligned(__alignof__(struct inotify_event)))); - const struct inotify_event *ev; + const struct inotify_event *ev = NULL; char path[PATH_MAX + 1]; bool found = false; ssize_t n; |