Skip to content
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.

Sign up for GitHub

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

Jump to bottom

lxc config trust dumps stack #192

Closed
raharper opened this issue Feb 9, 2015 · 0 comments · Fixed by #194
Closed

lxc config trust dumps stack #192

raharper opened this issue Feb 9, 2015 · 0 comments · Fixed by #194

Comments

@raharper
Copy link

raharper commented Feb 9, 2015

Exploring the cli, I ran lxc config trust and it dumped stack.

Reproduce:

  1. get lxd, deps and build it with make, then run
    ~/go/bin/lxd --debug --tcp 127.0.0.1:8443
  2. ~/go/bin/lxc config trust

Error Output:
% ~/go/bin/lxc config trust
panic: runtime error: index out of range

goroutine 16 [running]:
runtime.panic(0x736880, 0xb2db9c)
/usr/lib/go/src/pkg/runtime/panic.c:279 +0xf5
main.(*configCmd).run(0xc208000c10, 0xc20804c340, 0xc208000c80, 0x1, 0x1, 0x0, 0x0)
/home/rharper/go/src/github.com/lxc/lxd/lxc/config.go:52 +0xf88
main.run(0x0, 0x0)
/home/rharper/go/src/github.com/lxc/lxd/lxc/main.go:63 +0x74e
main.main()
/home/rharper/go/src/github.com/lxc/lxd/lxc/main.go:15 +0x26

goroutine 19 [finalizer wait]:
runtime.park(0x420990, 0xb43230, 0xb2fe49)
/usr/lib/go/src/pkg/runtime/proc.c:1369 +0x89
runtime.parkunlock(0xb43230, 0xb2fe49)
/usr/lib/go/src/pkg/runtime/proc.c:1385 +0x3b
runfinq()
/usr/lib/go/src/pkg/runtime/mgc0.c:2644 +0xcf
runtime.goexit()
/usr/lib/go/src/pkg/runtime/proc.c:1445

goroutine 17 [syscall]:
runtime.goexit()
/usr/lib/go/src/pkg/runtime/proc.c:1445
tych0 pushed a commit to tych0/lxd that referenced this issue Feb 10, 2015
Fix a few places where we use args without checking for them
5795080 
Closes canonical#192

Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
@tych0 tych0 mentioned this issue Feb 10, 2015
Merged
@1wxyz 1wxyz mentioned this issue Jun 13, 2019
Closed
simondeziel pushed a commit to simondeziel/lxd that referenced this issue Nov 3, 2023
@stgraber @simondeziel
incusd/instance/qemu: Add NVME disk support
f40f4d9 
Closes canonical#192

Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
simondeziel pushed a commit to simondeziel/lxd that referenced this issue Nov 6, 2023
@stgraber @simondeziel
lxd/instance/qemu: Add NVME disk support
f87ee49 
Closes canonical#192

Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix a few places where we use args without checking for them tych0/lxd
1 participant
@raharper