Skip to content

Check in Len Before Index#251

Merged
davidmdm merged 1 commit intoyokecd:mainfrom
dandeandean:fix-oob-read
Mar 16, 2026
Merged

Check in Len Before Index#251
davidmdm merged 1 commit intoyokecd:mainfrom
dandeandean:fix-oob-read

Conversation

@dandeandean
Copy link
Contributor

Hello, I've recently discovered yoke and I love the idea of being able to define my deployments in a real language.

Either way, I've been playing around with the yoke CLI & I noticed a bug right off the bat: yoke atc crashes if you don't have any airway and you attempt to view yaml.

I figured the PR was so small I might as well just put it up instead of posting an issue. Let me know what you think.

@dandeandean dandeandean changed the title Check in bounds before index Check in Bounds Before Index Mar 15, 2026
@dandeandean dandeandean changed the title Check in Bounds Before Index Check in Len Before Index Mar 15, 2026
@davidmdm
Copy link
Collaborator

Hey thanks so much for your contribution.

The yoke ATC command is a part that I haven't had time to focus on recently. All help is appreciated.

For instance I would love to add Flight/ClusterFlight support to that command, and improve some rough edges around the UI.

Thanks once more! I'll make sure to merge this in tonight!

@davidmdm davidmdm merged commit c2c71b8 into yokecd:main Mar 16, 2026
3 checks passed
@dandeandean dandeandean deleted the fix-oob-read branch March 17, 2026 00:12
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