sandbox cleanup

Gary E. Miller gem at rellim.com
Thu Jun 1 22:59:45 UTC 2017


Yo Hal!

On Thu, 01 Jun 2017 15:48:54 -0700
Hal Murray via devel <devel at ntpsec.org> wrote:

> The root of the problem is that APIs are complicated if things like 
> malloc/free are done across API boundaries.  I think we should fix
> the API so that sandbox never frees its arguments.

Now you know why Rust was invented.  As long as C is used there will
always be confusion on what a pointer points to.

> The cast to gid_t doesn't seem necessary.  (static gid_t sw_gid;)

Depends on the OS, remove the cast and you'll get warnings.

The cast was added in commit 6791d614ca5128882e6e16bb695c17c052ba5591 to
fix warnings.  Checking git log is usually the fastest way to answer many
qustions.

RGDS
GARY
---------------------------------------------------------------------------
Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97703
	gem at rellim.com  Tel:+1 541 382 8588

	    Veritas liberabit vos. -- Quid est veritas?
    "If you can’t measure it, you can’t improve it." - Lord Kelvin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ntpsec.org/pipermail/devel/attachments/20170601/a54c9a4e/attachment.bin>


More information about the devel mailing list