Fix addrman crashes
Fix addrman crashes

A function returned the element to remove from a bucket, instead of its
position in that bucket. This function was only called when a tried
bucket overflowed, which only happens after many outgoing connections
have been made.

Closes: #1065, #1156

syscoin/syscoin,Infernoman/crowncoin,Earlz/dobbscoin-source,Crowndev/crowncoin,Earlz/dobbscoin-source,wiggi/fairbrix-0.6.3,syscoin/syscoin,Crowndev/crowncoin,Earlz/dobbscoin-source,dobbscoin/dobbscoin-source,syscoin/syscoin,Earlz/dobbscoin-source,domob1812/crowncoin,dobbscoin/dobbscoin-source,Crowndev/crowncoin,wiggi/fairbrix-0.6.3,Infernoman/crowncoin,Infernoman/crowncoin,dobbscoin/dobbscoin-source,domob1812/crowncoin,syscoin/syscoin,wiggi/fairbrix-0.6.3,Crowndev/crowncoin,wiggi/fairbrix-0.6.3,Infernoman/crowncoin,Crowndev/crowncoin,dobbscoin/dobbscoin-source,domob1812/crowncoin,syscoin/syscoin,domob1812/crowncoin,wiggi/fairbrix-0.6.3,Earlz/dobbscoin-source,syscoin/syscoin,syscoin/syscoin,dobbscoin/dobbscoin-source,Infernoman/crowncoin,Crowndev/crowncoin,domob1812/crowncoin,domob1812/crowncoin,Infernoman/crowncoin,syscoin/syscoin,Earlz/dobbscoin-source,dobbscoin/dobbscoin-source