fix(gnovm): Fix panic when calling len()/cap() on pointer array

[notJoon]

Description

closes #2707

The GetLength(), GetCapacity() method in TypedValue has been updated to handle pointer type.

Contributors' checklist...

• Added new tests, or not needed, or not feasible
• Provided an example (e.g. screenshot) to aid review or the PR is self-explanatory
• Updated the official documentation or not needed
• No breaking changes were made, or a BREAKING CHANGE: xxx message was included in the description
• Added references to related issues and PRs
• Provided any useful hints for running manual tests
• Added new benchmarks to generated graphs, if any. More info here.

[codecov]

Codecov Report

Attention: Patch coverage is 58.33333% with 15 lines in your changes missing coverage. Please review.

Project coverage is 60.84%. Comparing base (11d29a2) to head (044054d).
Report is 4 commits behind head on master.

Files with missing lines	Patch %	Lines
gnovm/pkg/gnolang/values.go	58.33%	12 Missing and 3 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2709      +/-   ##
==========================================
- Coverage   60.85%   60.84%   -0.01%     
==========================================
  Files         563      563              
  Lines       75169    75190      +21     
==========================================
+ Hits        45741    45749       +8     
- Misses      26062    26073      +11     
- Partials     3366     3368       +2     

Flag	Coverage Δ
contribs/gnodev	60.65% <ø> (-0.82%)	⬇️
contribs/gnofaucet	14.46% <ø> (ø)
gno.land	67.21% <ø> (ø)
gnovm	65.63% <58.33%> (+0.04%)	⬆️
misc/genstd	80.54% <ø> (ø)
misc/logos	19.88% <ø> (-0.36%)	⬇️
tm2	62.05% <ø> (-0.07%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[MikaelVallenet]

Hello, @notJoon thanks for the fix 🙏, Can you fix the cap function that suffers from the same problem too

func main() {
	exp := [...]string{"HELLO"}
	x := cap(&exp)
	println(x)
}

[notJoon]

Hello, @notJoon thanks for the fix 🙏, Can you fix the cap function that suffers from the same problem too

modified the cap function in the same way. I had forgotten this function. thanks @MikaelVallenet

a089650

[deelawn]

Thanks for the fix Joon. Can you also make these changes in the if tv.V == nil blocks so that something like this won't crash the VM?

package main

func main() {
	printLen(nil)
}

func printLen(arr *[2]int) {
	println(len(arr))
}

[notJoon]

Can you also make these changes in the if tv.V == nil blocks so that something like this won't crash the VM?

I just added a PointerType handler for the array case when tv.V is nil in the GetLength and GetCapacity functions. Thanks for catch @deelawn 👍

[deelawn]

Nice work 💪