Skip to content

Update compile flags per security review #4368

@eseidelGoogle

Description

@eseidelGoogle

During security review we were asked to make sure our C++ compile flags for Release configs use the following:

Compile-time flags: -O2 -D_FORTIFY_SOURCE=2 -fstack-protector-all -fPIE -Wa,--noexecstack -Wformat -Wformat-security
Link-time flags: -pie -Wl,-z,relro,-z,now

This is non-urgent, but we should address for 1.0.

FYI @abarth @chinmaygarde

Metadata

Metadata

Assignees

No one assigned

    Labels

    P2Important issues not at the top of the work lista: releaseChallenges faced when attempting to productionize an appc: contributor-productivityTeam-specific productivity, code health, technical debt.engineflutter/engine related. See also e: labels.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions