Fix potential nil pointer dereference in container event monitoring by Nepomuk5665 · Pull Request #13551 · docker/compose
The condition for checking container restart state had incorrect operator precedence. The expression: inspect.State != nil && inspect.State.Restarting || inspect.State.Running is evaluated as: (inspect.State != nil && inspect.State.Restarting) || inspect.State.Running This means if inspect.State is nil and inspect.State.Restarting is false (which would trigger a panic), the code would attempt to access inspect.State.Running, causing a nil pointer dereference. This fix adds parentheses to ensure the nil check applies to both state checks: inspect.State != nil && (inspect.State.Restarting || inspect.State.Running) Signed-off-by: Nepomuk Crhonek <105591323+Nepomuk5665@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters