DiamondLightSource / DiamondLightSource/ppmac_read_csv

Add protected region

Open
#2 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
2
Forks
0
PR merge metrics
No merged PRs in 30d

Description

Add protection region value and check it against base address.

As minimum, the protected region should be the first 8 bytes, following the manual recommendation.
It might be interesting more restriction than that, due to the usage of ushm for simulated motors.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the C code that handles the base address and assigns data into the Power PMAC buffer. Read how the buffer address is currently checked, then define the protected region as the first 8 bytes and verify that the base address cannot overlap it. Confirm the behavior with the available project checks, though no test file is named in the issue.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
embedded-iot
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.