hyperion-v4l2: taking screenshot results in memory access error

[Thinner77]

• [x] I confirm that this is an issue rather than a question.

Bug report

Using hyperion-v4l2 (2.0.17-beta.1) to take a screenshot results in memory access error. hyperiond is running fine.

I don't think it's related, but brightness, contrast, saturation is supposedly set to 0.

v4l2-ctl -d /dev/video0 --list-ctrls User Controls

                 brightness 0x00980900 (int)    : min=0 max=1023 step=1 default=448 value=448 flags=slider
                   contrast 0x00980901 (int)    : min=0 max=1023 step=1 default=464 value=464 flags=slider
                 saturation 0x00980902 (int)    : min=0 max=1023 step=1 default=512 value=512 flags=slider
                        hue 0x00980903 (int)    : min=-3583 max=3583 step=1 default=0 value=0 flags=slider
                  sharpness 0x0098091b (int)    : min=0 max=255 step=1 default=96 value=96 flags=slider

Steps to reproduce

./hyperion-v4l2 --device /dev/video0 --input 0 --video-standard PAL --signal-detection-disabled --screenshot --debug hyperion-v4l2: Version : 2.0.17-beta.1 (master (GitHub-49c346e0/d5438acb-1712087086)) build time: Apr 12 2024 20:14:04 2024-04-12T20:16:54.062 || V4L2 : Grabber.cpp:142:setWidthHeight() | Set new width: 640, height: 480 for capture 2024-04-12T20:16:54.067 || V4L2 : Set flipmode to NO_CHANGE 2024-04-12T20:16:54.077 || V4L2 : V4L2Grabber.cpp:151:init() | Set device (path) to: usbtv (/dev/video0) 2024-04-12T20:16:54.078 || V4L2 : Grabber.cpp:142:setWidthHeight() | Set new width: 720, height: 576 for capture 2024-04-12T20:16:54.080 || V4L2 : V4L2Grabber.cpp:488:init_device() | Set device input to: Composite 2024-04-12T20:16:54.093 |__| V4L2 : V4L2Grabber.cpp:507:init_device() | Video standard=PAL 2024-04-12T20:16:54.093 || V4L2 : V4L2Grabber.cpp:614:init_device() | Set resolution to width=720 height=576 2024-04-12T20:16:54.095 || V4L2 : V4L2Grabber.cpp:651:init_device() | Set brightness to 0 2024-04-12T20:16:54.096 |__| V4L2 : V4L2Grabber.cpp:660:init_device() | Set contrast to 0 2024-04-12T20:16:54.097 || V4L2 : V4L2Grabber.cpp:669:init_device() | Set saturation to 0 2024-04-12T20:16:54.097 |__| V4L2 : V4L2Grabber.cpp:707:init_device() | Pixel format=YUYV QObject::connect(EncoderThreadManager, V4L2Grabber): invalid nullptr parameter Speicherzugriffsfehler

What is expected?

screenshot is saved

What is actually happening?

hyperion-v4l2 crashes

System

Hyperion Server:

• Build: master (GitHub-49c346e0/d5438acb-1712087086)
• Build time: Apr 12 2024 20:12:23
• Git Remote: https://github.com/hyperion-project/hyperion.ng.git
• Version: 2.0.17-beta.1
• UI Lang: de (BrowserLang: de)
• UI Access: expert
• Avail Screen Cap.: dispmanx,framebuffer,qt
• Avail Video Cap.: v4l2
• Avail Audio Cap.: audio
• Avail Services: boblight,cec,effectengine,forwarder,flatbuffer,protobuffer,mDNS,SSDP,borderdetection
• Config path: /home/pi/.hyperion
• Database: read/write
• Mode: Non-GUI

Hyperion Server OS:

• Distribution: Raspbian GNU/Linux 11 (bullseye)
• Architecture: arm
• CPU Model: ARMv7 Processor rev 4 (v7l)
• CPU Type: Raspberry Pi Zero 2 W Rev 1.0
• CPU Revision: 902120
• CPU Hardware: BCM2835
• Kernel: linux (6.1.21-v7+ (WS: 32))
• Root/Admin: false
• Qt Version: 5.15.2
• Python Version: 3.9.2
• Browser: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:109.0) Gecko/20100101 Firefox/115.0

[Lord-Grey]

@Thinner77 Thanks for reporting. This is a confirmed bug. We will look into it.

[Thinner77]

Thank you, works.