codesenberg / bombardier

Fast cross-platform HTTP benchmarking tool written in Go
MIT License
6.03k stars 315 forks source link

Panic starting 1.02 Windows 386 version #18

Closed morganbr closed 7 years ago

morganbr commented 7 years ago

The Windows 386 1.02 build fails to start sending requests with the output below. The x64 version seems to work for me. I'm running Windows 10 x64 and using the exes from the releases.

> bombardier-windows-386.exe http://localhost:5000/json
Bombarding http://localhost:5000/json for 10s using 125 connections
[-----------------------------------------------------------------------------]
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x0 pc=0x44cddc]

goroutine 71 [running]:
sync/atomic.AddUint64(0x1119ac94, 0x98, 0x0, 0x1, 0x69d57d)
        C:/Go/src/sync/atomic/asm_386.s:112 +0xc
main.(*bombardier).writeStatistics(0x1119ac30, 0x1b, 0x0, 0x98, 0x0, 0xc8, 0xb3b0, 0x0)
        C:/Users/frei/Documents/Go/Workspaces/Default/src/codesenberg/bombardier/bombardier.go:140 +0x59
main.(*bombardier).performSingleRequest(0x1119ac30)
        C:/Users/frei/Documents/Go/Workspaces/Default/src/codesenberg/bombardier/bombardier.go:167 +0xe1
main.(*bombardier).worker(0x1119ac30)
        C:/Users/frei/Documents/Go/Workspaces/Default/src/codesenberg/bombardier/bombardier.go:172 +0x3c
main.(*bombardier).bombard.func1(0x1119ac30)
        C:/Users/frei/Documents/Go/Workspaces/Default/src/codesenberg/bombardier/bombardier.go:234 +0x48
created by main.(*bombardier).bombard
        C:/Users/frei/Documents/Go/Workspaces/Default/src/codesenberg/bombardier/bombardier.go:235 +0xed
codesenberg commented 7 years ago

Thanks for the report. I think I know what the problem is. Gonna release v1.03 with the fix for this in a day or two, maybe, even today.

codesenberg commented 7 years ago

@morganbr, check out v1.0.3 it should be fixed now.

morganbr commented 7 years ago

@codesenberg that fixed it. Thanks!