gerrit.automotivelinux Code Review - AGL/meta-agl-devel.git/blob

   1 From 357634ceae6d730131b26bc3e72b4cf0b3e902ad Mon Sep 17 00:00:00 2001
   2 From: Marius Vlad <marius.vlad@collabora.com>
   3 Date: Mon, 19 Jun 2023 10:34:44 +0300
   4 Subject: [PATCH] shell/wayland/display: Handle flags as bitfield entries
   5
   6 We can have preferred and current mode or'ed in the same time, so we
   7 can't really check just for current mode. Make sure we do get a valid
   8 width/height for the outputs to avoid going further.
   9
  10 Bug-AGL: SPEC-4841
  11 Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
  12 ---
  13  shell/wayland/display.cc | 4 +++-
  14  1 file changed, 3 insertions(+), 1 deletion(-)
  15
  16 diff --git a/shell/wayland/display.cc b/shell/wayland/display.cc
  17 index 434dd05..5ace67c 100644
  18 --- a/shell/wayland/display.cc
  19 +++ b/shell/wayland/display.cc
  20 @@ -258,12 +258,14 @@ void Display::display_handle_mode(void* data,
  21    (void)flags;
  22    auto* oi = static_cast<output_info_t*>(data);
  23
  24 -  if (flags == WL_OUTPUT_MODE_CURRENT) {
  25 +  if ((flags & WL_OUTPUT_MODE_CURRENT) == WL_OUTPUT_MODE_CURRENT) {
  26      oi->height = static_cast<unsigned int>(height);
  27      oi->width = static_cast<unsigned int>(width);
  28      oi->refresh_rate = refresh;
  29    }
  30
  31 +  assert(oi->height && oi->width);
  32 +
  33    FML_DLOG(INFO) << "Video mode: " << width << " x " << height << " @ "
  34                   << (refresh > 1000 ? refresh / 1000.0 : (double)refresh)
  35                   << " Hz";
  36 --
  37 2.35.1
  38