Comparison at arp.c line 283 is wrong.

Bug #1668339 reported by markus5
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
parprouted (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

The comparison at arp.c line 283 is wrong. In 64-bit machines, a long is 64 bits. The cast is wrong. The s_addr is 32 bits long.

It should be something like this.

if ( ipaddr.s_addr == ((struct in_addr *) cur_entry->req_frame.arp.arp_tpa)->s_addr && ifindex != cur_entry->req_if.sll_ifindex ) {

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package parprouted - 0.70-3

---------------
parprouted (0.70-3) unstable; urgency=medium

  * Standards-Version updated.
  * Always cast to 32 bits. (LP: #1668339)

 -- Luciano Bello <email address hidden> Mon, 24 Jul 2017 21:51:12 -0400

Changed in parprouted (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.