net/rpc/jsonrpc: client nil pointer deference on invalid reply

[gopherbot]

by nos.adrian:

Returning invalid data to the client can result in a nil pointer deference:

http://play.golang.org/p/FAGJ4upO4Z

Unit test & proposed fix attached.

Attachments:

1. jsonrpc.patch (1600 bytes)

[ianlancetaylor]

Comment 1:

Labels changed: added go1.1maybe.

[robpike]

Comment 2:

Thanks. Fix looks right to me.
You are not on the official Go contributors list. The fix is tiny so I'm happy to make
it myself if you want to avoid the minor bureaucracy that entails, or you can go through
the official channels by following the instructions at
http://golang.org/doc/contribute.html
Please advise.

Labels changed: added priority-later, go1.1, removed priority-triage, go1.1maybe.

Status changed to WaitingForReply.

[gopherbot]

Comment 3 by nos.adrian:

I think I managed all the steps required. issue #7691045 should be available for review.

[rsc]

Comment 4:

This issue was closed by revision 0d559f7.

Status changed to Fixed.