On Sep 27, 10:15pm, bou...@antioche.eu.org (Manuel Bouyer) wrote:
-- Subject: Re: CVS commit: src/sys/arch/xen/xen
| > if (newstart != start) {
| > printf("uvm_map didn't give us back our vm space\n");
| > + uvm_unmap1(map, newstart, newstart + size, 0);
| > + i
On Sat, Sep 27, 2014 at 11:51:59AM -0400, Christos Zoulas wrote:
> On Sep 27, 8:36am, m...@m00nbsd.net (Maxime Villard) wrote:
> -- Subject: Re: CVS commit: src/sys/arch/xen/xen
>
> | One however returns an error without freeing:
> |
> | if (newstart != start) {
> | printf("uvm_m
Module Name:src
Committed By: christos
Date: Sun Sep 21 17:11:07 UTC 2014
Modified Files:
src/sys/dev/pci: twa.c
Log Message:
fix memset size inconsistency
This fix is wrong.
- memset(p, sizeof(struct twa_param_9k *), 10);
+ memset(p, 0, sizeof(*p));
p is
On Sep 27, 8:36am, m...@m00nbsd.net (Maxime Villard) wrote:
-- Subject: Re: CVS commit: src/sys/arch/xen/xen
| One however returns an error without freeing:
|
| if (newstart != start) {
| printf("uvm_map didn't give us back our vm space\n");
| return EINVAL;
|
On Sep 27, 3:44pm, m...@m00nbsd.net (Maxime Villard) wrote:
-- Subject: Re: CVS commit: src/sys/dev/pci
| This fix is wrong.
|
| - memset(p, sizeof(struct twa_param_9k *), 10);
| + memset(p, 0, sizeof(*p));
|
| p is a [10] array, your memset only initializes the first pointer.
|
| Seri
On Saturday 27 Sep 2014 11:04:09 Alan Barrett wrote:
> On Fri, 26 Sep 2014, Roy Marples wrote:
> >>>Log Message:
> >>>Remove \$ as a hidden marker as vis(3) wasn't setting it
> >>>and it clobbered VIS_SHELL | VIS_CSTYLE.
> >>>
> >> This is wrong. "vis -l" outputs \$, and with this change,
> >> unv
On Fri, 26 Sep 2014, Roy Marples wrote:
Log Message:
Remove \$ as a hidden marker as vis(3) wasn't setting it
and it clobbered VIS_SHELL | VIS_CSTYLE.
This is wrong. "vis -l" outputs \$, and with this change,
unvis won't correctly handle it.
unvis is not intended to reverse shell-style esca
Le 21/09/2014 20:22, Christos Zoulas a écrit :
On Sep 21, 7:57pm, m...@m00nbsd.net (Maxime Villard) wrote:
-- Subject: Re: CVS commit: src/sys/arch/xen/xen
| Did you test this change? Verily I'm not sure it's a proper bug, but I
| kept it in my list so that someone (bouyer@?) could investigate.