Skip to content

Fix security and robustness issues, add release automation - #230

Merged
kpchoi merged 9 commits into
mainfrom
kp_handling_issues_20260713
Jul 22, 2026
Merged

Fix security and robustness issues, add release automation#230
kpchoi merged 9 commits into
mainfrom
kp_handling_issues_20260713

Conversation

@kpchoi

@kpchoi kpchoi commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

This PR addresses multiple security and robustness issues and adds release
automation:

Bug Fixes

  • Prevent signed integer overflow in VLC exp-golomb decoding by using unsigned
    arithmetic with masked shifts (no branch penalty)
  • Add buffer overflow protection in bitstream reading by checking read_size
    limit
  • Validate TransCoeff range in DC and AC coefficient parsing
  • Fix buffer size validation in decoder with s64 casting to prevent overflow

Release Automation

  • Add GitHub Actions workflow to automatically generate SHA256SUMS for
    releases
  • Add GitHub Actions workflow to verify release tag version matches oapv.h
    version numbers

Issues Resolved

kpchoi added 7 commits July 13, 2026 18:15
Signed-off-by: KP Choi <kp5.choi@samsung.com>
Signed-off-by: KP Choi <kp5.choi@samsung.com>
Signed-off-by: KP Choi <kp5.choi@samsung.com>
Signed-off-by: KP Choi <kp5.choi@samsung.com>
Signed-off-by: KP Choi <kp5.choi@samsung.com>
Signed-off-by: KP Choi <kp5.choi@samsung.com>
@kpchoi
kpchoi requested review from mss-park and tuzm24 July 22, 2026 06:30

@tuzm24 tuzm24 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

kpchoi added 2 commits July 22, 2026 15:40
Signed-off-by: KP Choi <kp5.choi@samsung.com>
Signed-off-by: KP Choi <kp5.choi@samsung.com>
@kpchoi
kpchoi merged commit 3f0c355 into main Jul 22, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants