Hi,
I have been working on a diff to do canaries in a better way.
Canaries (enabled by the C malloc option) are values stored after the
requested size that are checked for being overwritten on calling
free(3). At the moment we only do this for chunks (sub-page sized
allocations).
To be able to
Thanks, that is apparently better than I suggested and reasonable.
And I confirmed it can also avoid the issue.
I appreciate if this is applied.
And, yes I believe compilation error is bug of compiler, not source code.
I posted this compilation error to Intel C++ compiler forum on Sunday.
https://
Brent Cook wrote:
> Why not just make the variable type match the return type to begin with?
sure, that's reasonable.
>
> --- a/src/lib/libtls/tls_util.c
> +++ b/src/lib/libtls/tls_util.c
> @@ -105,7 +105,8 @@ tls_load_file(const char *name, size_t *len, char
> *password)
> FILE *fp;
>
On Sun, Oct 02, 2016 at 02:30:11PM -0700, Philip Guenther wrote:
> On Sun, 2 Oct 2016, Rafael Zalamena wrote:
> > This diff is an improvement and an attempt to fix the bug where the
> > ntpd(8) not always stays running.
> >
> > During the review of syslogd fork+exec diff I noticed the use of dup3
On Sun, 2 Oct 2016, Rafael Zalamena wrote:
> This diff is an improvement and an attempt to fix the bug where the
> ntpd(8) not always stays running.
>
> During the review of syslogd fork+exec diff I noticed the use of dup3()
> and went to read its man page: dup2() doesn't always remove the CLOEX
On Sat, Oct 01, 2016 at 07:41:13PM +0200, Rafael Zalamena wrote:
> This could be replaced with "closefrom(4);".
Updated diff:
- use closefrom(2)
- use execvp(3) to allow starting syslogd(8) without full path
- add a debug message to make testing easier
ok?
bluhm
Index: usr.sbin/syslogd/privsep.
This diff is an improvement and an attempt to fix the bug where the ntpd(8)
not always stays running.
During the review of syslogd fork+exec diff I noticed the use of dup3()
and went to read its man page: dup2() doesn't always remove the CLOEXEC
flag from the descriptor, so using dup3() is a bette
Just like the switch(4) diff, this one does the same thing for the
bridge(4).
This diff removes bridge(4) code from if.c and uses the detach hook to
be notified about interface removals.
ok?
Index: net/if.c
===
RCS file: /home/obsd
mpi@ suggested that it would be possible to use if_detachhooks to handle
the interface teardown instead of adding code to if.c, so this diff does
exactly that.
Not only we get to remove switch(4) code from if.c, we also get less lines
of code by removing some duplicated teardown procedure in
switc
On Sat, Oct 01, 2016 at 06:01:34PM +0200, Martin Natano wrote:
> After committing the new MNT_NOPERM flag I got some complaints that my
> code doesn't work by people that recompiled mount_ffs, but didn't reboot
> to the new kernel. I don't blame them; in that situation sys_mount()
> silently ignore
> From: "Theo de Raadt"
> Date: Fri, 30 Sep 2016 09:38:30 -0600
>
> > Diff below is a possible way to fix this. But in a way we're cheating
> > here since we'll still consume more than 2047 bytes of stack space
> > when we descend into wskbd_initmute(). So perhaps we should rewrite
> > this cod
I'm trying to figure out where the regression in IPv6/NDP is and here's
what I found so far.
When the route expiration time got converted from time_second to
time_uptime we forgot to do the same for values inside RAs. I'm not
sure what's the real impact of this, but it is clearly wrong. Diff
bel
Found a bunch of these in my /var/log/messages:
Oct 2 18:30:58 idefix make: vfprintf %s NULL in "Applying :%c to "%s" "
Oct 2 18:30:58 idefix make: vfprintf %s NULL in "Result is "%s" "
To reproduce, issue 'make -n -d v' in a simple port, e.g. sysutils/cpuid
Index: usr.bin/make/varmodifiers.c
Only set the highspeed bit in bus_clock if highspeed is supported
by the controller. Needed as the bus_clock callback is called with
SDMMC_TIMING_HIGHSPEED even if the capability is not set.
Required to raise the bus width on pandaboard which doesn't have
the highspeed capability.
As anything oth
On Sun, Oct 02, 2016 at 08:50:39AM -0500, Brent Cook wrote:
> On Sat, Oct 1, 2016 at 7:12 PM, Ted Unangst wrote:
>
> > Kinichiro Inoguchi wrote:
> > > I would like to cast the return variable explicitly in tls_load_file().
> > > This fix also avoiding Intel C++ compiler "assertion failed" describe
On Sat, Oct 1, 2016 at 7:12 PM, Ted Unangst wrote:
> Kinichiro Inoguchi wrote:
> > I would like to cast the return variable explicitly in tls_load_file().
> > This fix also avoiding Intel C++ compiler "assertion failed" described
> here.
> > https://github.com/libressl-portable/portable/issues/20
ok
On Sun, Oct 2, 2016 at 12:34 AM, David Gwynne wrote:
> this gets rid of a linker warning.
>
> the impression i get is that EVP_CipherFinal_ex explicitely does
> not do cleanup of the cipher context thing, while EVP_CipherFinal
> could if it wanted to.
>
> ok?
>
> Index: usm.c
> ==
i think the change to move tcp timers to timeouts got this bit wrong.
we do want to print the timer if it is pending, it doesnt make sense
otherwise.
ok?
Index: trpt.c
===
RCS file: /cvs/src/usr.sbin/trpt/trpt.c,v
retrieving revisio
On Sat, Oct 1, 2016 at 11:58 PM, David Gwynne wrote:
> at the moment its relying on a pf header.
>
> ok?
ok guenther@
...though the set of #includes in that .h file should be reviewed:
lots of the .c file there pull in net/if.h, so why is snmpd.h doing
so, etc.
On Sat, Oct 01, 2016 at 11:00:07PM BST, Frederic Cambus wrote:
> Hi tech@,
>
> This diff adds missing airports and area codes for Poland, and fixes
> some erroneous entries.
Hi Frederic,
I've been meaning to do it for a while - thanks! :^)
A couple of comments below.
Cheers,
Raf
> OK?
>
> I
20 matches
Mail list logo