davecheney / pc

CLI tool to strip-mine the papercall.io api
7 stars 8 forks source link

pc speakers segfaults #2

Open arschles opened 5 years ago

arschles commented 5 years ago

Found with a build from 0181dbfd66a746aee369629485ec47518f1c29b2

Repro steps:

  1. run pc refresh
  2. run pc speakers

Result:

➜  pc git:(master) ✗ ./pc speakers
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x1306a3c]

goroutine 1 [running]:
main.speakers(0x6a5)
    /Users/arschles/src/pc/speakers.go:32 +0x2ac
main.main()
    /Users/arschles/src/pc/pc.go:43 +0xec0
davecheney commented 5 years ago

I see the problem. That report will not run correctly until we reveal the speakers names after shortlisting.

On Fri, 18 Jan 2019 at 11:40, Aaron Schlesinger notifications@github.com wrote:

Found with a build from 0181dbf

Repro steps:

run pc refresh run pc speakers

Result:

➜ pc git:(master) ✗ ./pc speakers

panic: runtime error: invalid memory address or nil pointer dereference

[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x1306a3c]

goroutine 1 [running]:

main.speakers(0x6a5)

/Users/arschles/src/pc/speakers.go:32 +0x2ac

main.main()

/Users/arschles/src/pc/pc.go:43 +0xec0

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.