# Re: \[PATCH v3 02/11\] reftable: fix resource leak in error path

- From: Han-Wen Nienhuys \<hidden\>
- Date: 2021-12-13 16:44:59 
- List: git
- Page: https://ratatoskr.run/git/2021/12/7915645
- Thread: https://ratatoskr.run/git/2021/12/7914155/t.md

```text
On Mon, Dec 13, 2021 at 5:21 PM Ævar Arnfjörð Bjarmason
[ref] wrote:
> Other things in the codebase don't check for NULL before feeding things
> to reftable_free(), and our own free() has a coccicheck rule to catch
> this sort of code, we should probably add reftable_free to that list...

Thanks, changed.

> > +     struct reftable_block_source source = { NULL };
>
> Nit: It doesn't matter for semantics, but usually we use "{ 0 }", and
> your 1/11 does too. Would be better to do that here for consistency.

I got a past review where someone complained about this. I don't mind
either way, but would rather not flipflop.

>
> > +     for (i = 0; i < sizeof(message)-1; i++) {
> > +             message[i] = (uint8_t)(rand() % 64 + ' ');
> > +     }
>
> style: braces not needede.

Done.

-- 
Han-Wen Nienhuys - Google Munich
I work 80%. Don't expect answers from me on Fridays.
--

Google Germany GmbH, Erika-Mann-Strasse 33, 80636 Munich

Registergericht und -nummer: Hamburg, HRB 86891

Sitz der Gesellschaft: Hamburg

Geschäftsführer: Paul Manicle, Halimah DeLaine Prado
```