Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 31 Aug 2008 13:05:44 +0100
From:      Dieter <freebsd@sopwith.solgatos.com>
Cc:        freebsd-firewire@freebsd.org
Subject:   Re: New and improved? patch 
Message-ID:  <200808312005.UAA08072@sopwith.solgatos.com>
In-Reply-To: Your message of "Sun, 31 Aug 2008 12:34:34 BST."

next in thread | raw e-mail | index | archive | help
> 7.0 AMD64 # ./fwcontrol -f -5
> fwcontrol: main:set_root_node out of range: No such file or directory
> 
> "No such file or directory" seems wrong

err(EX_USAGE, "%s:set_root_node out of range", __func__);

Err() is correct for places where errno would be set, such as
checking the return code from read(2).  But for the range checks,
errno does not apply, so err() gives misleading results.

The err(3) man page isn't clear, but it looks like errx(3) is
the function you want for the range checks.

Changing err() to errx() gives:

7.0 AMD64 # ./fwcontrol -f 70
fwcontrol: main:set_root_node out of range



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200808312005.UAA08072>