Re: git: 419822b372f5 - main - libgeom(3): Use calloc instead of malloc and bzero.

2022-03-11 Thread Mateusz Piotrowski
On 11/03/2022 15:15, Warner Losh wrote: On Fri, Mar 11, 2022, 4:10 AM Hans Petter Selasky wrote: On 3/11/22 11:57, Mateusz Piotrowski wrote: > Hi, > > I grepped out tree and there are many other places, which could be > converted to using calloc. Is there anything to watch

Re: git: 419822b372f5 - main - libgeom(3): Use calloc instead of malloc and bzero.

2022-03-11 Thread Ed Maste
On Fri, 11 Mar 2022 at 06:10, Hans Petter Selasky wrote: > > I don't see any issues converting malloc, memset/bzero sequences, as > long as they zero the full memory area allocated. In case someone starts picking up more of these, the more usual way to write it is "calloc(1, size)"

Re: git: 419822b372f5 - main - libgeom(3): Use calloc instead of malloc and bzero.

2022-03-11 Thread Warner Losh
On Fri, Mar 11, 2022, 4:10 AM Hans Petter Selasky wrote: > On 3/11/22 11:57, Mateusz Piotrowski wrote: > > Hi, > > > > I grepped out tree and there are many other places, which could be > > converted to using calloc. Is there anything to watch out for when > > converting malloc and memset call pa

Re: git: 419822b372f5 - main - libgeom(3): Use calloc instead of malloc and bzero.

2022-03-11 Thread Hans Petter Selasky
On 3/11/22 11:57, Mateusz Piotrowski wrote: Hi, I grepped out tree and there are many other places, which could be converted to using calloc. Is there anything to watch out for when converting malloc and memset call pairs to calloc? Hi, If you have a tool for it combined with a review, sh

Re: git: 419822b372f5 - main - libgeom(3): Use calloc instead of malloc and bzero.

2022-03-11 Thread Mateusz Piotrowski
Hi, I grepped out tree and there are many other places, which could be converted to using calloc. Is there anything to watch out for when converting malloc and memset call pairs to calloc? Best, Mateusz On 11/03/2022 10:42, Hans Petter Selasky wrote: The branch main has been updated by hsela

git: 419822b372f5 - main - libgeom(3): Use calloc instead of malloc and bzero.

2022-03-11 Thread Hans Petter Selasky
The branch main has been updated by hselasky: URL: https://cgit.FreeBSD.org/src/commit/?id=419822b372f543b22d7fb04eae0dffacf058feb6 commit 419822b372f543b22d7fb04eae0dffacf058feb6 Author: Hans Petter Selasky AuthorDate: 2022-03-10 09:16:27 + Commit: Hans Petter Selasky CommitDate: