<feed xmlns='http://www.w3.org/2005/Atom'>
<title>wireguard-openbsd/games/hack/hack.pager.c, branch jd/simplify-queueing</title>
<subtitle>WireGuard implementation for the OpenBSD kernel</subtitle>
<id>https://git.zx2c4.com/wireguard-openbsd/atom/games/hack/hack.pager.c?h=jd%2Fsimplify-queueing</id>
<link rel='self' href='https://git.zx2c4.com/wireguard-openbsd/atom/games/hack/hack.pager.c?h=jd%2Fsimplify-queueing'/>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/'/>
<updated>2019-06-28T13:32:41Z</updated>
<entry>
<title>When system calls indicate an error they return -1, not some arbitrary</title>
<updated>2019-06-28T13:32:41Z</updated>
<author>
<name>deraadt</name>
<email>deraadt@openbsd.org</email>
</author>
<published>2019-06-28T13:32:41Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=df69c215c7c66baf660f3f65414fd34796c96152'/>
<id>urn:sha1:df69c215c7c66baf660f3f65414fd34796c96152</id>
<content type='text'>
value &lt; 0.  errno is only updated in this case.  Change all (most?)
callers of syscalls to follow this better, and let's see if this strictness
helps us in the future.
</content>
</entry>
<entry>
<title>execl(3) calls must have the last argument casted to (char *)NULL</title>
<updated>2016-03-15T19:56:20Z</updated>
<author>
<name>mestre</name>
<email>mestre@openbsd.org</email>
</author>
<published>2016-03-15T19:56:20Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=64f0fa1112a15500ade44b4c96d7e6540ea7c90c'/>
<id>urn:sha1:64f0fa1112a15500ade44b4c96d7e6540ea7c90c</id>
<content type='text'>
At the time when this was commited me and tb@ discussed that it shouldn't be
changed, but still snucked in within a larger diff that we didn't notice.

OK tb@
</content>
</entry>
<entry>
<title>Remove several casts to (char *) 0 and replace them by NULL</title>
<updated>2016-01-09T21:54:11Z</updated>
<author>
<name>mestre</name>
<email>mestre@openbsd.org</email>
</author>
<published>2016-01-09T21:54:11Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=6acf840eb12048fe6e815d917fa237f8a82441d8'/>
<id>urn:sha1:6acf840eb12048fe6e815d917fa237f8a82441d8</id>
<content type='text'>
Prompted and OK by tb@
</content>
</entry>
<entry>
<title>This diff is rather large, but it's just simply removing unnecessary headers</title>
<updated>2016-01-09T18:33:15Z</updated>
<author>
<name>mestre</name>
<email>mestre@openbsd.org</email>
</author>
<published>2016-01-09T18:33:15Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=aed906e4b20d9afbda31247cdb6acf6f29da8819'/>
<id>urn:sha1:aed906e4b20d9afbda31247cdb6acf6f29da8819</id>
<content type='text'>
, remove some lint-style comments, add missing void to functions without args,
also some function prototypes (extern) were missing their args, so I added them
as they were defined in hack.h

OK tb@ on separate diffs
</content>
</entry>
<entry>
<title>Annotate funcs with __attribute__((printf(...))) and clean up the fallout:</title>
<updated>2015-09-27T05:13:11Z</updated>
<author>
<name>guenther</name>
<email>guenther@openbsd.org</email>
</author>
<published>2015-09-27T05:13:11Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=911134d2940395eae97932b9f9c69e48c962c4de'/>
<id>urn:sha1:911134d2940395eae97932b9f9c69e48c962c4de</id>
<content type='text'>
 * lots of foo(str) --&gt; foo("%s", str) transformations
 * one totally insane foo(fmt, ap) --&gt; vfoo(fmt, ap) conversion: how did
   this ever work?
 * prefer const char[] over char* for static format strings, as it lets
   gcc check the format and eliminates an unnecessary pointer

ok beck@
</content>
</entry>
<entry>
<title>Match bwrite()'s prototype with write()'s, zapping a lint comment</title>
<updated>2014-03-11T08:05:15Z</updated>
<author>
<name>guenther</name>
<email>guenther@openbsd.org</email>
</author>
<published>2014-03-11T08:05:15Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=ebf3d589a8c26b74d169ae56dc1cd5127550a23c'/>
<id>urn:sha1:ebf3d589a8c26b74d169ae56dc1cd5127550a23c</id>
<content type='text'>
Delete casts to char* of arguments to bwrite() and free()
</content>
</entry>
<entry>
<title>Use the help pager code, but repair it in various ways.</title>
<updated>2011-03-16T18:11:37Z</updated>
<author>
<name>deraadt</name>
<email>deraadt@openbsd.org</email>
</author>
<published>2011-03-16T18:11:37Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=357d2205e9b1b4a829500734eea7258e41fb8d85'/>
<id>urn:sha1:357d2205e9b1b4a829500734eea7258e41fb8d85</id>
<content type='text'>
(I had to fix the $PAGER support beyond the patch supplied by Simon)
from Simon.Nicolussi@student.uibk.ac.at
</content>
</entry>
<entry>
<title>rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and</title>
<updated>2009-10-27T23:59:19Z</updated>
<author>
<name>deraadt</name>
<email>deraadt@openbsd.org</email>
</author>
<published>2009-10-27T23:59:19Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=043fbe51c197dbbcd422e917b65f765d8b5f8874'/>
<id>urn:sha1:043fbe51c197dbbcd422e917b65f765d8b5f8874</id>
<content type='text'>
unmaintainable).  these days, people use source.  these id's do not provide
any benefit, and do hurt the small install media
(the 33,000 line diff is essentially mechanical)
ok with the idea millert, ok dms
</content>
</entry>
<entry>
<title>remove unused variable</title>
<updated>2009-06-03T14:47:47Z</updated>
<author>
<name>chl</name>
<email>chl@openbsd.org</email>
</author>
<published>2009-06-03T14:47:47Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=148cf60f0bdb8cdcf4f2347ee7f5399de08ec163'/>
<id>urn:sha1:148cf60f0bdb8cdcf4f2347ee7f5399de08ec163</id>
<content type='text'>
ok millert@ ray@
</content>
</entry>
<entry>
<title>Fix fgets handling.</title>
<updated>2009-06-01T22:57:14Z</updated>
<author>
<name>ray</name>
<email>ray@openbsd.org</email>
</author>
<published>2009-06-01T22:57:14Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=8c6fe208668137bcd073676b0056ae4721d31d7e'/>
<id>urn:sha1:8c6fe208668137bcd073676b0056ae4721d31d7e</id>
<content type='text'>
1. Skip blank lines instead of dereferencing NULL-1.
2. If bufr is a blank line don't pass bufr+1 as a pointer.

OK millert
</content>
</entry>
</feed>
