Not necessary in this pull request, but I suggest making icmpa_send return void. I don't think the caller should ever look at its return value.