kotovalexarian-likes-github/moby--moby
1
0
Fork 0
mirror of https://github.com/moby/moby.git synced 2022-11-09 12:21:53 -05:00
Code Releases Activity

Merge pull request #43839 from thaJeztah/journald_fix_break

Browse source 
logger/journald: fix SA4011: ineffective break statement
This commit is contained in:
Sebastiaan van Stijn 2022-07-20 18:46:11 +02:00 • committed by GitHub
commit 7624f8aeb1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
daemon/logger/journald/read.go
View file

@ -227,6 +227,7 @@ func (s *journald) followJournal(logWatcher *logger.LogWatcher, j *C.sd_journal,
waitTimeout := C.uint64_t(250000) // 0.25s waitTimeout := C.uint64_t(250000) // 0.25s
LOOP:
for { for {
status := C.sd_journal_wait(j, waitTimeout) status := C.sd_journal_wait(j, waitTimeout)
if status < 0 { if status < 0 {
@ -235,7 +236,7 @@ func (s *journald) followJournal(logWatcher *logger.LogWatcher, j *C.sd_journal,
} }
select { select {
case <-logWatcher.WatchConsumerGone(): case <-logWatcher.WatchConsumerGone():
break // won't be able to write anything anymore break LOOP // won't be able to write anything anymore
case <-s.closed: case <-s.closed:
// container is gone, drain journal // container is gone, drain journal
default: default: