New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pod: fix segfault when there are no arguments to inspect #2684
pod: fix segfault when there are no arguments to inspect #2684
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: giuseppe The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Closes: containers#2681 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
3f37280
to
7fee7d8
Compare
LGTM |
@tobwin PTAL |
@@ -44,6 +44,11 @@ func podInspectCmd(c *cliconfig.PodInspectValues) error { | |||
pod *adapter.Pod |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't comment up higher, but the message at line 37 says: "Act on the latest container podman is aware of". I think that should be "latest pod podman".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM, but I think I spotted a cut/paste error a few lines up. Could you verify that please?
/lgtm |
Inspired by containers#2684, I wrote a CI test to look for other such instances in which a command is invoked without a required argument. 'podman umount' seems to be the only one. Signed-off-by: Ed Santiago <santiago@redhat.com>
Inspired by containers#2684, I wrote a CI test to look for other such instances in which a command is invoked without a required argument. 'podman umount' seems to be the only one. Signed-off-by: Ed Santiago <santiago@redhat.com>
Inspired by containers#2684, I wrote a CI test to look for other such instances in which a command is invoked without a required argument. 'podman umount' seems to be the only one; but 'podman mount' also needs a change, to indicate that its CONTAINER argument is optional. Signed-off-by: Ed Santiago <santiago@redhat.com>
Inspired by containers#2684, I wrote a CI test to look for other such instances in which a command is invoked without a required argument. 'podman umount' seems to be the only one, and solution is simple: checkAllAndLatest() already does the check for us. Resolve a few other problems uncovered by testing: podman mount: indicate that CONTAINER arg is optional podman pod stats: ditto podman generate kube: remove check for -l (latest) flag, it isn't actually implemented. Signed-off-by: Ed Santiago <santiago@redhat.com>
Inspired by containers#2684, I wrote a CI test to look for other such instances in which a command is invoked without a required argument. 'podman umount' seems to be the only one, and solution is simple: checkAllAndLatest() already does the check for us. Resolve a few other problems uncovered by testing: podman mount: indicate that CONTAINER arg is optional podman pod stats: ditto podman generate kube: remove check for -l (latest) flag, it isn't actually implemented. Signed-off-by: Ed Santiago <santiago@redhat.com>
Closes: #2681
Signed-off-by: Giuseppe Scrivano gscrivan@redhat.com