Add Apache LICENSE

Add a license terms for inarp.

Change-Id: I7a182e9cd62dfe51f8e0d83ab00119bfe4388afb
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

Spelling fixes

Spelling errors found using github.com/lucasdemarchi/codespell
A tool to fix common misspellings.
This tool is licensed under GNU General Public License, version 2.

Change-Id: I7e7db

Spelling fixes

Spelling errors found using github.com/lucasdemarchi/codespell
A tool to fix common misspellings.
This tool is licensed under GNU General Public License, version 2.

Change-Id: I7e7db9a0d49b59f998e68ea805fa806bac37de71
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>

show more ...

Add MAINTAINERS file

Change-Id: I80a51760496cdcb7ca7acc0be03dc58cc6c04c36
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

Convert build process to autotools

Replaced the use of a manual Makefile with the use of autotools to
automatically verify and generate the necessary build files. Follow the
steps outlined within th

Convert build process to autotools

Replaced the use of a manual Makefile with the use of autotools to
automatically verify and generate the necessary build files. Follow the
steps outlined within the README.md file to build the package.

Change-Id: If5d1cf54ab6f3609b5e65b361b6b389ddce5377d
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>

show more ...

Avoid signed and unsigned comparison

The 'NLMSG_OK' macro uses a _u32 struct entry for comparison but the
length from recv is signed. Passing the signed type directly into the
macro causes a signed

Avoid signed and unsigned comparison

The 'NLMSG_OK' macro uses a _u32 struct entry for comparison but the
length from recv is signed. Passing the signed type directly into the
macro causes a signed and unsigned comparison warning.

Fixes openbmc/inarp#3.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

show more ...

Makefile: Append instead of overwriting CFLAGS.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

Makefile: Do not override CC.

The makefile currently forces CC = gcc, which prevents building against
a Yocto SDK.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Signed-off-by: Jeremy Kerr <jk

Makefile: Do not override CC.

The makefile currently forces CC = gcc, which prevents building against
a Yocto SDK.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

show more ...

Use syslog for log output

We should be logging to syslog rather than stdout/stderr

This change implements logging to syslog (via vsyslog()), and adds a
couple of arguments to control debug behaviou

Use syslog for log output

We should be logging to syslog rather than stdout/stderr

This change implements logging to syslog (via vsyslog()), and adds a
couple of arguments to control debug behaviour.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

show more ...

Use netlink for link state queries

Rather than do a once-off query for interfaces at boot, we should be
using netlink to query for initial link states, and listen for link
update & deletion events.

Use netlink for link state queries

Rather than do a once-off query for interfaces at boot, we should be
using netlink to query for initial link states, and listen for link
update & deletion events.

This change adds functionality to receive RTM_NEWLINK and RTM_DELLINK
messages, and updates the interfaces array using that.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

show more ...

Rename inarp_ctx.socket

We want to introduce another socket descriptor, so give this one a more
specific name.

No functional change to code.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

Makefile: Add clean & install targets

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

Listen on all interfaces

Rather than requiring an interface to listen on on the command-line,
just listen on all interface, and use the sockaddr returned from
recvfrom() to determine which interface

Listen on all interfaces

Rather than requiring an interface to listen on on the command-line,
just listen on all interface, and use the sockaddr returned from
recvfrom() to determine which interface the inarp request came from.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

show more ...

Prepare for multiple-interface support

We want to support inarp on any system interface. This change prepares
for that by splitting the interface into into a separate array.

We only have one elemen

Prepare for multiple-interface support

We want to support inarp on any system interface. This change prepares
for that by splitting the interface into into a separate array.

We only have one element in that array at present.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

show more ...

Use a context structure to encapsulate lifetime context data

We'll want to be adding more local context in an upcoming change, so
prepare for that by moving daemon-lifetime contect to a single
struc

Use a context structure to encapsulate lifetime context data

We'll want to be adding more local context in an upcoming change, so
prepare for that by moving daemon-lifetime contect to a single
structure.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

show more ...

Use strncpy when populating struct ifreq->ifr_name

We already check that ifname will fit into ifr_name in main(), but in
case do_ifreq gets called from elsewhere, use strncpy() instead of
strcpy().

Use strncpy when populating struct ifreq->ifr_name

We already check that ifname will fit into ifr_name in main(), but in
case do_ifreq gets called from elsewhere, use strncpy() instead of
strcpy().

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

show more ...

Use a strong type for ethernet MAC addresses

Instead of using a generic array, use a strict struct type for ethernet
addresses, as is done with IP addresses.

Signed-off-by: Jeremy Kerr <jk@ozlabs.o

Use a strong type for ethernet MAC addresses

Instead of using a generic array, use a strict struct type for ethernet
addresses, as is done with IP addresses.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

show more ...

Consolidate MAC-address printing to a separate function

We have two (different) paths to print an ethernet MAC address.
This change consolidates them into a single function to convert address
bytes

Consolidate MAC-address printing to a separate function

We have two (different) paths to print an ethernet MAC address.
This change consolidates them into a single function to convert address
bytes to a string, and uses that for our printf()s.

Suggested by Joel Stanley <joel@jms.id.au>.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

show more ...

Constify pointer-typed function arguments

Most of our pointer function arguments are inputs; mark them as const.

Based on a suggestion from Joel Stanley <joel@jms.id.au>.

Signed-off-by: Jeremy Ker

Constify pointer-typed function arguments

Most of our pointer function arguments are inputs; mark them as const.

Based on a suggestion from Joel Stanley <joel@jms.id.au>.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

show more ...

Reorder #includes

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

Minor whitespace cleanup

Remove tabs in the header comment, reformat indentation where it's
non-standard.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

Rename socket_address to addr

... it's not really the address of a socket.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

Remove unnecessary casts to (void *)

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

Add copyright attribution for IBM

We've done a considerable re-write, so add copyright info for IBM.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

Print local address in output

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

Consolidate & clarify variable definitions

Move main()'s variable defintions together, and clarify "src_mac" (which
actually means local).

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>