Consider enabling stack-smash protection by default

esp-rs / esp-template

A minimal esp-hal application template for use with cargo-generate

Apache License 2.0

136 stars 27 forks source link

Consider enabling stack-smash protection by default #146

Open bjoernQ opened 6 months ago

bjoernQ commented 6 months ago

Since we support it, we could add "-Z", "stack-protector=all", to .cargo/config.toml to get stack-smash protection (and kind of stack overflow protection in our case). Probably with a hint like we use for forcing stack frames.

Otherwise, the feature is quite hard to discover (but will be mentioned in no-std training in a future PR)

MabezDev commented 6 months ago

I think this is a good idea, the only down side is nightly is that is needed :(