Panic on some regex #3

New Issue

Ghost · 2023-07-26T04:54:03Z

Ghost commented

2023-07-26 04:54:03 +00:00

import (
	"go.arsenm.dev/pcre"
	"testing"
)

func TestRegex(t *testing.T) {
	regex := pcre.MustCompile(`varnish(?: \(Varnish\/([\d.]{1,250})\))?`)
	regex.FindStringSubmatch("1.1 varnish")
}

=== RUN TestRegex
--- FAIL: TestRegex (209.38s)
panic: runtime error: slice bounds out of range [:18446744073709551615] with capacity 16 [recovered]
panic: runtime error: slice bounds out of range [:18446744073709551615] with capacity 16

r.match returns an invalid array

```go import ( "go.arsenm.dev/pcre" "testing" ) func TestRegex(t *testing.T) { regex := pcre.MustCompile(`varnish(?: \(Varnish\/([\d.]{1,250})\))?`) regex.FindStringSubmatch("1.1 varnish") } ``` === RUN TestRegex --- FAIL: TestRegex (209.38s) panic: runtime error: slice bounds out of range [:18446744073709551615] with capacity 16 [recovered] panic: runtime error: slice bounds out of range [:18446744073709551615] with capacity 16 `r.match` returns an invalid array ![image](/attachments/fe2dd63f-bc9a-4e5c-bf95-564965bede84)

image.png

46 KiB

Elara6331 commented

2023-07-26 05:37:40 +00:00

This should be fixed in 6d906d55a6. Can you please test it and let me know if it works?

This should be fixed in 6d906d55a6d26dfab6d7ecf76b01acdd2caa124f. Can you please test it and let me know if it works?

Ghost commented

2023-07-26 07:24:27 +00:00

@Elara6331 It works! Thank you!

Ghost closed this issue

2023-07-26 07:24:28 +00:00

Elara6331 commented