On 2 September 2022 02:09:23 GMT+03:00, Jessica Zhang <quic_jesszhan@xxxxxxxxxxx> wrote:
>From: Rohith Iyer <quic_rohiiyer@xxxxxxxxxxx>
>
>Fix null pointer deference caused by drmModeAtomicReq being
>allocated before set_property was called when modetest was run
>with the atomic flag.
... being allocated _after_ ...
Other than that:
Reviewed-by: Dmitry Baryshkov<dmitry.baryshkov@xxxxxxxxxx>
>
>Reviewed-by: Rob Clark <robdclark@xxxxxxxxx>
>Signed-off-by: Rohith Iyer <quic_rohiiyer@xxxxxxxxxxx>
>Signed-off-by: Jessica Zhang <quic_jesszhan@xxxxxxxxxxx>
>---
> tests/modetest/modetest.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
>diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
>index 42e2d1f4..2c31c4fc 100644
>--- a/tests/modetest/modetest.c
>+++ b/tests/modetest/modetest.c
>@@ -2186,11 +2186,13 @@ int main(int argc, char **argv)
> dump_resource(&dev, planes);
> dump_resource(&dev, framebuffers);
>
>+ if (dev.use_atomic)
>+ dev.req = drmModeAtomicAlloc();
>+
> for (i = 0; i < prop_count; ++i)
> set_property(&dev, &prop_args[i]);
>
> if (dev.use_atomic) {
>- dev.req = drmModeAtomicAlloc();
>
> if (set_preferred || (count && plane_count)) {
> uint64_t cap = 0;
--
With best wishes
Dmitry
