Hi

You state you want to
"if (the ipv4 stuff is defined) {do all the fun printing}".
However what you do is
if (the device is existing) { print all the IP4 and IP6-stuff }

However if the ip4-part isn't existing, you'll get the error you describe accessing it.

You might want to

• change the condition inside the loop (e.g. check if $nic->{ip} is defined)
• change the brackets of the if - I'm not sure what you want to print, however I assume you want to print the ip6-part always, but the ip4-part only if it is there

HTH, Rata