[functions][not_needed] Copy scope so it isn't overridden

An edge case found an ASAN use-after-free error where a scope value was
getting overridden by the evaluated result. This copies the scope in the
case where it would have been overridden.

Bug: crbug.com/gn/12
Change-Id: I08322fc73e6e459fbe4c5f35e7f5154ded74746c
Reviewed-on: https://gn-review.googlesource.com/c/2920
Reviewed-by: Scott Graham <scottmg@google.com>
Commit-Queue: Julie Hockett <juliehockett@google.com>
2 files changed
tree: d37d3ff09b6f10019bd930602dc33f721cf05d2d
  1. base/
  2. build/
  3. docs/
  4. infra/
  5. tools/
  6. util/
  7. .clang-format
  8. .editorconfig
  9. .gitignore
  10. .style.yapf
  11. AUTHORS
  12. LICENSE
  13. OWNERS
  14. README.md
README.md

GN

GN is a meta-build system that generates build files for Ninja. There is documentation in docs/.

Getting started

git clone https://gn.googlesource.com/gn
cd gn
python build/gen.py
ninja -C out
# To run tests:
out/gn_unittests

On Windows, it is expected that cl.exe, link.exe, and lib.exe can be found in PATH, so you'll want to run from a Visual Studio command prompt, or similar.

On Linux and Mac, the default compiler is clang++, a recent version is expected to be found in PATH. This can be overridden by setting CC, CXX, and AR.

Sending patches

GN uses Gerrit for code review. The short version of how to patch is:

Register at https://gn-review.googlesource.com.

... edit code ...
ninja -C out && out/gn_unittests

Then, to upload a change for review:

git commit
git cl upload --gerrit

When revising a change, use:

git commit --amend
git cl upload --gerrit

which will add the new changes to the existing code review, rather than creating a new one.

We ask that all contributors sign Google's Contributor License Agreement (either individual or corporate as appropriate, select ‘any other Google project’).

Community

You may ask questions and follow along w/ GN‘s development on Chromium’s gn-dev@ Google Group.