Skip to content

dev: add a note on what we consider to be an attack surface in AP#7359

Open
peterbarker wants to merge 1 commit intoArduPilot:masterfrom
peterbarker:pr/attack-surface
Open

dev: add a note on what we consider to be an attack surface in AP#7359
peterbarker wants to merge 1 commit intoArduPilot:masterfrom
peterbarker:pr/attack-surface

Conversation

@peterbarker
Copy link
Contributor

No description provided.

@rmackay9
Copy link
Contributor

Maybe we should put this on a new page? Maybe a page dedicated to security issues.. it could link to mavlink signing for example

@peterbarker
Copy link
Contributor Author

Maybe we should put this on a new page? Maybe a page dedicated to security issues.. it could link to mavlink signing for example

Open to the concept, but I thought this did belong under our "design decisions" thing. And yeah, should definitely be linking to mavlink signing if we have a page for that.

@Ryanf55
Copy link
Contributor

Ryanf55 commented Jan 26, 2026

I support the intent of this. Good idea.

@Hwurzburg
Copy link
Contributor

needs a reviewer approval from @rmackay9 since he had a suggeston

@rmackay9
Copy link
Contributor

rmackay9 commented Mar 9, 2026

I'll create a new OEM Security (or something like that) wiki page and we can include the advice there

@timtuxworth
Copy link
Contributor

timtuxworth commented Mar 9, 2026

As per today's Dev call discussion I agree it should be it's own page, but not necessarily "OEM" @rmackay9 - anyone might be interested in securing their platform even if not an OEM.

Perhaps "Security and Hardening Guide"

This should probably also talk about MAVLink signing, and the "lock down parameter changes" capability.


ArduPilot works in a resource-constrained environment. We can't afford all of the sanity checks that we might otherwise include.

To this end, we do not consider every input to the autopilot firmware potentially malicious. We trust our SPI-connected inertial sensors to be trusted, for example.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
To this end, we do not consider every input to the autopilot firmware potentially malicious. We trust our SPI-connected inertial sensors to be trusted, for example.
To this end, we do not consider every input to the autopilot firmware potentially malicious. We trust our SPI-connected inertial sensors to be well behaved, for example.

Just a little odd having "trust" in there twice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: ReadyForDevCall

Development

Successfully merging this pull request may close these issues.

6 participants