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

ubuntu release 0.7.3 crashes in WSL2 (ubuntu 20.04) #28

Closed
lopeg opened this issue Feb 8, 2023 · 7 comments
Closed

ubuntu release 0.7.3 crashes in WSL2 (ubuntu 20.04) #28

lopeg opened this issue Feb 8, 2023 · 7 comments
Assignees

Comments

@lopeg
Copy link

lopeg commented Feb 8, 2023

Expected Behavior

Successfully run
./azqr-ubuntu-latest-amd64 -s "XXX" -g "YYY"

Actual Behavior

`2023/02/08 13:03:35 Analyzing SQL in Resource Group YYY
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0xbfb1ff]

goroutine 86 [running]:
github.com/cmendible/azqr/internal/scanners.(*StorageScanner).Scan(0xc000382b40, {0x7fffaeff2edd, 0x1a})
/home/runner/work/azqr/azqr/internal/scanners/st.go:50 +0x19f
main.reviewRunner.func1(0xc000153d00, {0x7fffaeff2edd, 0x1a})
/home/runner/work/azqr/azqr/cmd/azqr/main.go:195 +0x102
created by main.reviewRunner
/home/runner/work/azqr/azqr/cmd/azqr/main.go:189 +0x73`

Steps to Reproduce the Problem

@cmendible
Copy link
Member

cmendible commented Feb 8, 2023

Hi it seems to be failing accessing the access tier of an Storage Account. Can you give me more info on the particular storage accounts?

cmendible added a commit that referenced this issue Feb 8, 2023
@cmendible
Copy link
Member

@lopeg please download and try azqr from here: https://github.com/cmendible/azqr/actions/runs/4125206893 and let me know if it fixes the issue.

@pamontag
Copy link

pamontag commented Feb 8, 2023

With the new build (windows) I have no more the st.go error and the assessment was completed. Thank you

image

@cmendible
Copy link
Member

Great! I will release 0.7.4.

Out of curiosity is the Storage Account an v1 or ADLS?

@pamontag
Copy link

pamontag commented Feb 8, 2023

image

@cmendible
Copy link
Member

v1 was the culprit.

Thanks for the feedback!

@lopeg
Copy link
Author

lopeg commented Feb 9, 2023

@lopeg please download and try azqr from here: https://github.com/cmendible/azqr/actions/runs/4125206893 and let me know if it fixes the issue.

Fixed. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants