[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/6] scripts/coccinelle: add error-use-after-free.cocci
From: |
Peter Maydell |
Subject: |
Re: [PATCH 1/6] scripts/coccinelle: add error-use-after-free.cocci |
Date: |
Tue, 31 Mar 2020 19:56:34 +0100 |
On Tue, 31 Mar 2020 at 10:01, Markus Armbruster <address@hidden> wrote:
> I doubt including basic spatch instructions with every script is a good
> idea. Better explain it in one place, with proper maintenance.
> scripts/coccinelle/README? We could be a bit more verbose there,
> e.g. to clarify required vs. suggested options.
I find it useful -- you (hopefully) get the actual command line
the original author used, rather than having to guess which
options might be significant. (eg the last one I added uses
--keep-comments --smpl-spacing --include-headers --dir target
which aren't all always used but which do all matter here for
getting the change that went in in the commit that used the script.)
Most of us use coccinelle as an occasional tool, not an
everyday one, so not having to look through the docs to figure
out the right rune is a benefit, even for the options that
we do use on pretty much every spatch run.
thanks
-- PMM
Re: [PATCH 2/6] block/mirror: fix use after free of local_err, Max Reitz, 2020/03/25
[PATCH 1/6] scripts/coccinelle: add error-use-after-free.cocci, Vladimir Sementsov-Ogievskiy, 2020/03/24
Re: [PATCH 1/6] scripts/coccinelle: add error-use-after-free.cocci,
Peter Maydell <=
[PATCH 4/6] migration/colo: fix use after free of local_err, Vladimir Sementsov-Ogievskiy, 2020/03/24
[PATCH 3/6] dump/win_dump: fix use after free of err, Vladimir Sementsov-Ogievskiy, 2020/03/24
[PATCH 5/6] migration/ram: fix use after free of local_err, Vladimir Sementsov-Ogievskiy, 2020/03/24
[PATCH 6/6] qga/commands-posix: fix use after free of local_err, Vladimir Sementsov-Ogievskiy, 2020/03/24