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

Missing input checks to GetInput on alt-da #11987

Open
anupsv opened this issue Sep 18, 2024 · 0 comments
Open

Missing input checks to GetInput on alt-da #11987

anupsv opened this issue Sep 18, 2024 · 0 comments

Comments

@anupsv
Copy link

anupsv commented Sep 18, 2024

Bug Description
There is no nil checks for the input comm CommitmentData which causes the panics when being used. Would be good to have checks against the input.

Steps to Reproduce

var commit op_plasma.CommitmentData
_, err := daClient.GetInput(ts.Ctx, commit)

Expected behavior
Should return an error as "MalformedData" or something similar

Environment Information:
Any environment

Configurations:
N/A

Logs:

panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x20 pc=0x1019f2fe8]

⚠️ Notice: Issues that do not include the following sections will be subject to closure:

  • Bug Description
  • Steps to Reproduce
  • Environment Information

Please ensure all required sections are filled out accurately to expedite the debugging process and improve issue resolution efficiency.

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

1 participant