slice bounds out of range error about hercules HOT 13 CLOSED

This is clearly a crash @Gr33nbl00d

Can you send me the repo you analyzed; if it is not possible, I will try to suggest fixes but I will need your assistance to test.

from hercules.

Hi no problem it is quite problematic to send you this repo as it is really big 1.7M Lines of Code and also im not allowed as it is confidential. Its is reproducable and crashes always on the same step with the same error. No Problem i will try to help :)

from hercules.

OK. If you are able to build Hercules, then I can send you a patch which prints the hashes of the two offending compared Git blobs, then you execute git show <hash> for each and send me the output (file contents) through email. Does it sound good?

from hercules.

Yes sounds good, but i have to admit i never worked with GO yet ;) so i have to fiddle a bit arround to find out how to compile it :)

from hercules.

@Gr33nbl00d Please use the binary from "debug" release, it contains the patch: https://github.com/src-d/hercules/releases/tag/debug

from hercules.

thank you i will try

from hercules.

ok the new output is:

hercules --devs . | python labours.py -m devs
2417 / 30036 2h13m9s2019/01/21 12:11:06
2019/01/21 12:11:06 ecc236816c8bddf3523d39ec0f7650591c829fb6
2019/01/21 12:11:06 7f06d8f55b010f2daf01d2b50747c2ba2ff11f8d
panic: runtime error: slice bounds out of range [recovered]
panic: runtime error: slice bounds out of range

goroutine 4846 [running]:
gopkg.in/src-d/hercules.v6/internal/plumbing.(*RenameAnalysis).blobsAreClose.func1(0xc004f24af0, 0xc005fa43c0)
c:/gopath/src/gopkg.in/src-d/hercules.v6/internal/plumbing/renames.go:367 +0x19d
panic(0xe8a9a0, 0xd64140)
C:/go/src/runtime/panic.go:513 +0x1c7
gopkg.in/src-d/hercules.v6/internal/plumbing.(*RenameAnalysis).blobsAreClose(0xc0009e8790, 0xc004f24af0, 0xc005fa43c0, 0xc00ac16700, 0x0, 0x0)
c:/gopath/src/gopkg.in/src-d/hercules.v6/internal/plumbing/renames.go:418 +0xb9f
gopkg.in/src-d/hercules.v6/internal/plumbing.(*RenameAnalysis).Consume.func1(0x0, 0x0)
c:/gopath/src/gopkg.in/src-d/hercules.v6/internal/plumbing/renames.go:232 +0x41b
gopkg.in/src-d/hercules.v6/internal/plumbing.(*RenameAnalysis).Consume.func3(0xc038909360, 0xc0009e98d0)
c:/gopath/src/gopkg.in/src-d/hercules.v6/internal/plumbing/renames.go:314 +0x2e
created by gopkg.in/src-d/hercules.v6/internal/plumbing.(*RenameAnalysis).Consume
c:/gopath/src/gopkg.in/src-d/hercules.v6/internal/plumbing/renames.go:314 +0x1a7b

No data has been read - has Hercules crashed?

from hercules.

Good, so now I need you to run

git show ecc236816c8bddf3523d39ec0f7650591c829fb6 | gzip > file1.gz
git show 7f06d8f55b010f2daf01d2b50747c2ba2ff11f8d | gzip > file2.gz

And send me those two files. My email can be taken from e.g. https://github.com/src-d/hercules/commit/b8fb564c3acd2da61027c9f909a22e33c9c1626b.patch

from hercules.

i send you a link with the files direct sending of files did not work because of an email gateway error i got.

Florian

from hercules.

I downloaded the files successfully, thanks. I will reproduce the problem this week and push the fix asap.

from hercules.

Ok no Problem take your time :) I am glad i could help and looking forward for your fix :)

from hercules.

@Gr33nbl00d Looks like I fixed the bug. I am publishing a new release later today.

from hercules.

Should be fixed in https://github.com/src-d/hercules/releases/tag/v7

from hercules.