Re: [patch] uvm_mmap / little change proposal

2018-08-21 Thread David CARLIER
I knew it was not documented for a reason but why is it ? Just curious ... but it s ok if we drop this change ... On Tue, 21 Aug 2018 at 16:38, Mark Kettenis wrote: > > > From: "Todd C. Miller" > > Date: Tue, 21 Aug 2018 09:19:05 -0600 > > > > On Tue, 21 A

Re: [patch] uvm_mmap / little change proposal

2018-08-21 Thread David CARLIER
No one else .. Linux has MAP_FIXED_NOREPLACE which is MAP_FIXED|MAP_EXCL equivalent... On Tue, 21 Aug 2018 at 16:19, Todd C. Miller wrote: > > On Tue, 21 Aug 2018 08:57:06 +0100, David CARLIER wrote: > > > For portability sake and to make it more "visible" I changed t

[patch] uvm_mmap / little change proposal

2018-08-21 Thread David CARLIER
Hi, For portability sake and to make it more "visible" I changed the __MAP_NOREPLACE to MAP_EXCL, documented it in the man page and also changed the errno to EEXIST instead (I thought it would reflect better the error in case the addressed mapped already exists but can be changed back in case ...)

Re: [update] base zlib

2018-07-14 Thread David CARLIER
Ah right if it relates to lib 1.2.8 that might do ... On Sat, 14 Jul 2018 at 19:41, Bryan Steele wrote: > > On Sat, Jul 14, 2018 at 07:32:23PM +0100, Stuart Henderson wrote: > > There have been several diffs to update this already, they have been > > rejected (upstream using some things that were

[patch] xargs - few allocation size fixes

2018-06-12 Thread David CARLIER
Hi, here a little patch proposal for xargs command line. Thanks. Kind regards. Index: xargs.c === RCS file: /cvs/src/usr.bin/xargs/xargs.c,v retrieving revision 1.33 diff -u -p -r1.33 xargs.c --- xargs.c 16 Oct 2017 13:10:50 - 1

Re: introduction of an additional non-POSIX function in libpthread

2017-07-18 Thread David CARLIER
Sure thing, here an updated diff. Regards. On 18 July 2017 at 16:22, Ted Unangst wrote: > David CARLIER wrote: > > Hi. > > > > I sent a diff originally to a smaller audience but finally decided to > post > > it in the mailing list. > > So it is to introduce t

Re: introduction of an additional non-POSIX function in libpthread

2017-07-18 Thread David CARLIER
Yes that s true NetBSD having a variadic like argument as well ... etc ... I did not have high expectations just wanted to propose ... :-) On 18 July 2017 at 14:55, Landry Breuil wrote: > On Tue, Jul 18, 2017 at 01:43:42PM +0100, David CARLIER wrote: > > Ah I recognise you you r

Re: introduction of an additional non-POSIX function in libpthread

2017-07-18 Thread David CARLIER
Oups mistake of mine. good catch On 18 July 2017 at 14:08, Karel Gardas wrote: > Why your patch defines MALLOC_STATS? > > -/* #define MALLOC_STATS */ > +#define MALLOC_STATS > > On Tue, Jul 18, 2017 at 2:43 PM, David CARLIER wrote: > > Ah I recognise you you re Mozilla

Re: introduction of an additional non-POSIX function in libpthread

2017-07-18 Thread David CARLIER
Ah I recognise you you re Mozilla contributor right ? You re probably right but that was just an example eventhough I admit it s not extremely widely used ... On 18 July 2017 at 13:36, Landry Breuil wrote: > On Tue, Jul 18, 2017 at 01:24:21PM +0100, David CARLIER wrote: > > Hi. > &g

introduction of an additional non-POSIX function in libpthread

2017-07-18 Thread David CARLIER
Hi. I sent a diff originally to a smaller audience but finally decided to post it in the mailing list. So it is to introduce the pthread_set_name_np's counterpart so pthread_get_name_np. Some softwares use it (NSPR for example) and anyway internally in my workplace we have multiplatform needing th

Re: Possible typo

2016-06-15 Thread David CARLIER
Ah ok I may have a not too updated mirror then I updated locally like 1h ago. Thanks ! On 15 June 2016 at 17:28, Todd C. Miller wrote: > On Wed, 15 Jun 2016 17:26:46 +0100, David CARLIER wrote: > > > maybe someone already mentioned it, but I tried to compile the kernel > today

Possible typo

2016-06-15 Thread David CARLIER
Hi all, maybe someone already mentioned it, but I tried to compile the kernel today and in sys/netinet/udp_usrreq.c, line 937 might contain an extra parenthesis after the first condition ie if ((inp->inp_flags & INP_IPSECFLOWINFO) != 0 *)* Hope it s useful. Regards.

[patch] installboot

2016-03-26 Thread David CARLIER
Hi, Here a little diff to fix small memory leaks inside installboot. Hope it finds its way. Kind regards. Index: i386_installboot.c === RCS file: /cvs/src/usr.sbin/installboot/i386_installboot.c,v retrieving revision 1.26 diff -u -p

Re: Little patch

2016-01-26 Thread David CARLIER
Nevermind otto yook care of it already :) On 26 January 2016 at 15:39, David CARLIER wrote: > Hi all, > > I just had a short chat with otto and would like to know if this little > patch would be considered. The reasoning is while debugging a program I got > a segmentation fault

Little patch

2016-01-26 Thread David CARLIER
Hi all, I just had a short chat with otto and would like to know if this little patch would be considered. The reasoning is while debugging a program I got a segmentation fault inside the loop as pool was NULL probably due to the fact that malloc_init was not called first. Thanks in advance. Kin

Re: MALLOC_STATS and pledge

2015-11-14 Thread David CARLIER
Might be indeed, there are other use cases with valgrind and protexec and so on ..., utrace might be a good suggestion too except maybe a need for an exception in pledge's side then. On 13 November 2015 at 18:34, Ted Unangst wrote: > David CARLIER wrote: >> Hi all, >> >

MALLOC_STATS and pledge

2015-11-13 Thread David CARLIER
Hi all, I ve tried to discuss this point with Otto Moerbeek but he might be very busy so I throw the topic here if you do not mind ... I have the habit to enable MALLOC_STATS but with the recent pledge feature, it s now difficult to debug some pledged applications with MALLOC_OPTIONS=D as, for ma

small memory leaks in localtime

2015-09-07 Thread David CARLIER
Hi all, Both internal state variables, lclptr and gmtptr are allocated once but never freed so I was wondering if this little diff which avoids the unnecessary dynamic allocations would have any use ? Thanks in advance. Regards. Index: localtime.c

small ed diff

2015-04-29 Thread David CARLIER
Hello everybody. There it is a tiny diff about ed ... 1/ using a long long (for 32 bits case) for strtol. 2/ realloc => reallocarray Hope it has any use Kind regards. Index: ed.h === RCS file: /cvs/src/bin/ed/ed.h,v retrieving

tiny tetris patch

2015-03-15 Thread David CARLIER
Hi all, As tetris is one of my preferred game :-) ... just did wrapper around setegid in same manner than xmalloc and such. If it can find any use ... Thanks. Index: scores.c === RCS file: /cvs/src/games/tetris/scores.c,v retrieving

Re: your mail

2015-01-02 Thread David Carlier
In OpenBSD we just write that as "free(mp)", the "if" isn't required. Will remember it ;-) Index: fuse.c === RCS file: /cvs/src/lib/libfuse/fuse.c,v retrieving revision 1.24 diff -u -p -r1.24 fuse.c --- fuse.c 20 May 2014 13:32:22 -

[no subject]

2015-01-02 Thread David Carlier
Hi again after discussins with Helg, second change is not relevant, so only the first should remains. Kind regards. Index: fuse.c === RCS file: /cvs/src/lib/libfuse/fuse.c,v retrieving revision 1.24 diff -u -p -r1.24 fuse.c --- fuse.

libfuse, potential memory leaks

2015-01-01 Thread David Carlier
Hello and happy new year. So this is a quite a small diff which, hopefully, will make lib fuse slightly better. Indeed, it might exist two small potential memory leaks, first the memory allocated after parsing command line when it succeeds, the other the very first option argument in the list. Hop

Re: ato* => strto*

2014-12-09 Thread David Carlier
t;, optarg, errstr); break; case 'f': todo |= FORKSTAT; @@ -224,8 +226,11 @@ main(int argc, char *argv[]) if (errstr) errx(1, "%s: %s", *argv, errstr); -if (*++argv) -reps = atoi(*argv); +if (*++argv) {

ato* => strto*

2014-12-09 Thread David Carlier
Hi all, I made a small first diff to replace some ato* functions call to strtonum/l ones ... Althought not sure about the sudo one because it looks contrib code and also might appear a bit "overengineered" ... but in case ! Thanks in advance. Index: bin/ps/ps.c ==

Small diff ...

2014-12-08 Thread David Carlier
Following small discussion I had with tedu@ this is just small diff, just for the sake of C standard "correctness" ... dropping some arithmetic pointer with void pointers ... Hopes it might find some interest/usefullness ... Thanks in advance. Index: gnu/gcc/gcc/unwind-dw2.c ===

Posix RealTime Extensions.

2014-11-16 Thread David Carlier
Hi all, sorry I m far to be an OpenBSD expert but I was wondering why it lacks some functions like sigqueue and such ... Is there any plan to add them in the future and if not what is the reasoning behind it ? Thanks in advance.

Posix Realtime extensions

2014-11-16 Thread David CARLIER
Hi all, sorry I m far to be an OpenBSD expert but I was wondering why it lacks some functions like sigqueue and such ... Is there any plan to add them in the future and if not what is the reasoning behind it ? Thanks in advance.

Re: 5.6 ... small detail

2014-11-04 Thread David Carlier
Thanks ! On 4 November 2014 19:00, Otto Moerbeek wrote: > On Tue, Nov 04, 2014 at 01:49:32PM -0500, Ted Unangst wrote: > > > On Tue, Nov 04, 2014 at 16:38, David Carlier wrote: > > > Hi all, > > > > > > Just updated to 5.6 and was happy to

5.6 ... small detail

2014-11-04 Thread David Carlier
Hi all, Just updated to 5.6 and was happy to see that gets(3) was removed for good ... althought it seems to be still available only in the man page. Kind regards.

Re: chmod.c undefined behavior

2014-09-24 Thread David Carlier
hwarze wrote: > Hi David, > > David Carlier wrote on Wed, Sep 24, 2014 at 10:19:57AM +0100: > > On 24 September 2014 10:10, Matti Karnaattu > wrote: > > >> I noticed that chmod.c have uninitialized variable char *ep > >> that was used. This diff clarify what I

Re: chmod.c undefined behavior

2014-09-24 Thread David Carlier
I mght be wrong, but why not just initialize to NULL? On 24 September 2014 10:10, Matti Karnaattu wrote: > I noticed that chmod.c have uninitialized variable char *ep that was > used. This diff clarify what I mean. > > > Index: chmod.c > =

Re: Small geoloc daemon

2014-08-10 Thread David Carlier
Hi Reyk told me about this and apologies for it. I am fixing it at the moment. Thanks for prompt feedback though. Kind regards. On 10 August 2014 14:19, Jérémie Courrèges-Anglas wrote: > David Carlier writes: > > > Hi all. > > > > I ve made a first draft of a new

Small geoloc daemon

2014-08-10 Thread David Carlier
it. Kind regards. David CARLIER Afilias Technologies

Re: Proposition

2014-07-11 Thread David Carlier
Fair point Reyk, I honestly did not think about this daemon approach ! Thanks for your inputs ! On 11 July 2014 11:59, Reyk Floeter wrote: > Hi, > > On Fri, Jul 11, 2014 at 11:33:19AM +0100, David Carlier wrote: > > I was wondering if a generic small geoloc lib might inter

Proposition

2014-07-11 Thread David Carlier
elayd) ... Kind regards. David CARLIER dotMobi / Afilias Technologies