Re: ber.c: Fix some minor issues in ober_read_element()

2022-11-02 Thread Martijn van Duren
On Wed, 2022-11-02 at 18:34 +0100, Claudio Jeker wrote: > On Wed, Nov 02, 2022 at 05:56:21PM +0100, Martijn van Duren wrote: > > On Wed, 2022-11-02 at 17:47 +0100, Claudio Jeker wrote: > > > On Wed, Nov 02, 2022 at 05:25:12PM +0100, Martijn van Duren wrote: > > > > On Wed, 2022-11-02 at 17:00

Re: ber.c: Fix some minor issues in ober_read_element()

2022-11-02 Thread Claudio Jeker
On Wed, Nov 02, 2022 at 05:56:21PM +0100, Martijn van Duren wrote: > On Wed, 2022-11-02 at 17:47 +0100, Claudio Jeker wrote: > > On Wed, Nov 02, 2022 at 05:25:12PM +0100, Martijn van Duren wrote: > > > On Wed, 2022-11-02 at 17:00 +0100, Claudio Jeker wrote: > > > > On Wed, Nov 02, 2022 at

Re: ber.c: Fix some minor issues in ober_read_element()

2022-11-02 Thread Martijn van Duren
On Wed, 2022-11-02 at 17:47 +0100, Claudio Jeker wrote: > On Wed, Nov 02, 2022 at 05:25:12PM +0100, Martijn van Duren wrote: > > On Wed, 2022-11-02 at 17:00 +0100, Claudio Jeker wrote: > > > On Wed, Nov 02, 2022 at 07:33:14AM +0100, Martijn van Duren wrote: > > > > I found 2 minor issues in the

Re: ber.c: Fix some minor issues in ober_read_element()

2022-11-02 Thread Claudio Jeker
On Wed, Nov 02, 2022 at 05:25:12PM +0100, Martijn van Duren wrote: > On Wed, 2022-11-02 at 17:00 +0100, Claudio Jeker wrote: > > On Wed, Nov 02, 2022 at 07:33:14AM +0100, Martijn van Duren wrote: > > > I found 2 minor issues in the handling of sequences/sets in > > > ober_read_element(): > > > 1)

Re: ber.c: Fix some minor issues in ober_read_element()

2022-11-02 Thread Martijn van Duren
On Wed, 2022-11-02 at 17:00 +0100, Claudio Jeker wrote: > On Wed, Nov 02, 2022 at 07:33:14AM +0100, Martijn van Duren wrote: > > I found 2 minor issues in the handling of sequences/sets in > > ober_read_element(): > > 1) An empty sequence/set (which is basically always) unconditionally > >

Re: ber.c: Fix some minor issues in ober_read_element()

2022-11-02 Thread Claudio Jeker
On Wed, Nov 02, 2022 at 07:33:14AM +0100, Martijn van Duren wrote: > I found 2 minor issues in the handling of sequences/sets in > ober_read_element(): > 1) An empty sequence/set (which is basically always) unconditionally >creates an (uninitialised) sub-element. Add the same length check >

ber.c: Fix some minor issues in ober_read_element()

2022-11-02 Thread Martijn van Duren
I found 2 minor issues in the handling of sequences/sets in ober_read_element(): 1) An empty sequence/set (which is basically always) unconditionally creates an (uninitialised) sub-element. Add the same length check used to check the next element 2) For each sub-element r is only checked for