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.

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

ACL command panics when asking for info of a non-existing user #3966

Closed
mangalaman93 opened this issue Sep 11, 2019 · 0 comments · Fixed by #3990
Closed

ACL command panics when asking for info of a non-existing user #3966

mangalaman93 opened this issue Sep 11, 2019 · 0 comments · Fixed by #3990
Assignees
Labels
area/crash Dgraph issues that cause an operation to fail, or the whole server to crash. area/enterprise Related to proprietary features exp/beginner Something most people could solve. kind/bug Something is broken. priority/P1 Serious issue that requires eventual attention (can wait a bit)
Milestone

Comments

@mangalaman93
Copy link
Member

What version of Dgraph are you using?

v1.1.0-6-g3e9810d6

Steps to reproduce the issue (command/config used to run Dgraph)

dgraph acl -a localhost:9080 info -u user1

Expected behaviour and actual result.

Expected: error that user doesn't exist
Actual: panic

[Decoder]: Using assembly version of decoder

Running transaction with dgraph endpoint: localhost:9180
Current password for groot:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0xa00798]

goroutine 1 [running]:
github.com/dgraph-io/dgraph/ee/acl.info(0xc000312a50, 0x191e6e0, 0xc0006e29c0)
	/home/aman/gocode/src/github.com/dgraph-io/dgraph/ee/acl/acl.go:668 +0x2f9
github.com/dgraph-io/dgraph/ee/acl.initSubcommands.func4(0xc000324fc0, 0xc0006fa580, 0x0, 0x4)
	/home/aman/gocode/src/github.com/dgraph-io/dgraph/ee/acl/run_ee.go:125 +0x32
github.com/dgraph-io/dgraph/vendor/github.com/spf13/cobra.(*Command).execute(0xc000324fc0, 0xc0006fa4c0, 0x4, 0x4, 0xc000324fc0, 0xc0006fa4c0)
	/home/aman/gocode/src/github.com/dgraph-io/dgraph/vendor/github.com/spf13/cobra/command.go:702 +0x289
github.com/dgraph-io/dgraph/vendor/github.com/spf13/cobra.(*Command).ExecuteC(0x22151a0, 0x0, 0x0, 0x0)
	/home/aman/gocode/src/github.com/dgraph-io/dgraph/vendor/github.com/spf13/cobra/command.go:783 +0x2ca
github.com/dgraph-io/dgraph/vendor/github.com/spf13/cobra.(*Command).Execute(...)
	/home/aman/gocode/src/github.com/dgraph-io/dgraph/vendor/github.com/spf13/cobra/command.go:736
github.com/dgraph-io/dgraph/dgraph/cmd.Execute()
	/home/aman/gocode/src/github.com/dgraph-io/dgraph/dgraph/cmd/root.go:68 +0x85
main.main()
	/home/aman/gocode/src/github.com/dgraph-io/dgraph/dgraph/main.go:33 +0x99
@mangalaman93 mangalaman93 self-assigned this Sep 11, 2019
@mangalaman93 mangalaman93 changed the title ACL command is panics when asking for info of a non-existing user ACL command panics when asking for info of a non-existing user Sep 11, 2019
@danielmai danielmai added the kind/bug Something is broken. label Sep 11, 2019
@campoy campoy added area/enterprise Related to proprietary features area/crash Dgraph issues that cause an operation to fail, or the whole server to crash. exp/beginner Something most people could solve. priority/P1 Serious issue that requires eventual attention (can wait a bit) labels Sep 13, 2019
@campoy campoy added this to the Dgraph v1.1.1 milestone Sep 13, 2019
campoy added a commit that referenced this issue Sep 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/crash Dgraph issues that cause an operation to fail, or the whole server to crash. area/enterprise Related to proprietary features exp/beginner Something most people could solve. kind/bug Something is broken. priority/P1 Serious issue that requires eventual attention (can wait a bit)
Development

Successfully merging a pull request may close this issue.

3 participants