I recommend that you end this function here because you called it make_arp (instead of send_arp). Also, it is better to reuse the send_packet function instead of redoing the same work that the function does