Back to build
Raw log | Switch to full mode | Login

'C:\b\s\w\ir\cipd_bin_packages\bin\python.exe' -u 'C:\b\s\w\ir\kitchen-checkout\depot_tools\recipes\recipe_modules\bot_update\resources\bot_update.py' --spec-path 'c:\b\s\w\ir\tmp\t\tmpcghjka' --revision_mapping_file 'c:\b\s\w\ir\tmp\t\tmpydrl96.json' --git-cache-dir 'C:\b\s\w\ir\cache\git' --cleanup-dir 'C:\b\s\w\ir\kitchen-workdir\recipe_cleanup\bot_update' --output_json 'c:\b\s\w\ir\tmp\t\tmpko6pcg.json' --revision boringssl@23e1a1f2d3c4ca0121eb93eb7d210501f9702317 in dir C:\b\s\w\ir\kitchen-workdir: allow_subannotations: False base_name: 'bot_update' cmd: ['C:\\b\\s\\w\\ir\\cipd_bin_packages\\bin\\python.exe', '-u', 'C:\\b\\s\\w\\ir\\kitchen-checkout\\depot_tools\\recipes\\recipe_modules\\bot_update\\resources\\bot_update.py', '--spec-path', 'c:\\b\\s\\w\\ir\\tmp\\t\\tmpcghjka', '--revision_mapping_file', 'c:\\b\\s\\w\\ir\\tmp\\t\\tmpydrl96.json', '--git-cache-dir', 'C:\\b\\s\\w\\ir\\cache\\git', '--cleanup-dir', 'C:\\b\\s\\w\\ir\\kitchen-workdir\\recipe_cleanup\\bot_update', '--output_json', 'c:\\b\\s\\w\\ir\\tmp\\t\\tmpko6pcg.json', '--revision', 'boringssl@23e1a1f2d3c4ca0121eb93eb7d210501f9702317'] env: {} env_prefixes: {} env_suffixes: {'PATH': 'C:\\b\\s\\w\\ir\\kitchen-checkout\\depot_tools'} infra_step: True name: 'bot_update' nest_level: 0 ok_ret: frozenset([0, 87, 88]) step_test_data: <lambda>(...) trigger_specs: () full environment: ALLUSERSPROFILE: C:\ProgramData APPDATA: C:\Users\chrome-bot\AppData\Roaming BOTO_CONFIG: C:\b\s\w\ir\tmp\gsutil_task\.boto BUILDBUCKET_EXPERIMENTAL: FALSE CIPD_CACHE_DIR: C:\b\s\cipd_cache\cache CIPD_EXE_SHIM: "C:\b\s\w\ir\cipd_bin_packages\vpython.exe" CIPD_PROTOCOL: v2 COMMONPROGRAMFILES: C:\Program Files\Common Files COMMONPROGRAMFILES(X86): C:\Program Files (x86)\Common Files COMMONPROGRAMW6432: C:\Program Files\Common Files COMPUTERNAME: WIN10-727F49A0- COMSPEC: C:\Windows\system32\cmd.exe DOCKER_CONFIG: C:\b\s\w\ir\tmp\docker_cfg_task DOCKER_TMPDIR: C:\b\s\w\ir\tmp\docker_tmp_task FPS_BROWSER_APP_PROFILE_STRING: Internet Explorer FPS_BROWSER_USER_PROFILE_STRING: Default GIT_CONFIG_NOSYSTEM: 1 GIT_TERMINAL_PROMPT: 0 HOMEDRIVE: C: HOMEPATH: \Users\chrome-bot INFRA_GIT_WRAPPER_HOME: C:\b\s\w\ir\tmp\git_home_task LC_ALL: English_United States.UTF-8 LC_COLLATE: English_United States.UTF-8 LC_CTYPE: English_United States.UTF-8 LC_MONETARY: English_United States.UTF-8 LC_NUMERIC: English_United States.UTF-8 LC_TIME: English_United States.UTF-8 LOCALAPPDATA: C:\Users\chrome-bot\AppData\Local LOGDOG_COORDINATOR_HOST: logs.chromium.org LOGDOG_STREAM_PREFIX: buildbucket/cr-buildbucket.appspot.com/8923053775805549904 LOGDOG_STREAM_PROJECT: boringssl LOGDOG_STREAM_SERVER_PATH: net.pipe:LUCILogDogKitchen_7292 LOGONSERVER: \\WIN10-727F49A0- LUCI_CONTEXT: C:\b\s\w\it9qxwla\luci_context.613493458 MAC_CHROMIUM_TMPDIR: C:\b\s\w\ir\tmp\t NO_GCE_CHECK: False NUMBER_OF_PROCESSORS: 8 ONEDRIVE: C:\Users\chrome-bot\OneDrive OS: Windows_NT PATH: C:\b\s\w\ir\cipd_bin_packages;C:\b\s\w\ir\cipd_bin_packages\bin;C:\b\s\cipd_cache\bin;C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Program Files\Puppet Labs\Puppet\bin;C:\b\s\w\ir\kitchen-checkout\depot_tools PATHEXT: .COM;.EXE;.BAT;.CMD;.VBS;.VBE;.JS;.JSE;.WSF;.WSH;.MSC PROCESSOR_ARCHITECTURE: AMD64 PROCESSOR_IDENTIFIER: Intel64 Family 6 Model 79 Stepping 0, GenuineIntel PROCESSOR_LEVEL: 6 PROCESSOR_REVISION: 4f00 PROGRAMDATA: C:\ProgramData PROGRAMFILES: C:\Program Files PROGRAMFILES(X86): C:\Program Files (x86) PROGRAMW6432: C:\Program Files PROMPT: $P$G PSMODULEPATH: C:\Program Files\WindowsPowerShell\Modules;C:\Windows\system32\WindowsPowerShell\v1.0\Modules PUBLIC: C:\Users\Public PYTHONIOENCODING: UTF-8 PYTHONUNBUFFERED: 1 SESSIONNAME: Console SWARMING_BOT_ID: win10-727f49a0-us-west1-b-94tq SWARMING_EXTERNAL_BOT_SETUP: 1 SWARMING_HEADLESS: 1 SWARMING_SERVER: https://chromium-swarm.appspot.com SWARMING_TASK_ID: 42af1de11ffe2a11 SYSTEMDRIVE: C: SYSTEMROOT: C:\Windows TEMP: C:\b\s\w\ir\tmp\t TEMPDIR: C:\b\s\w\ir\tmp\t TMP: C:\b\s\w\ir\tmp\t TMPDIR: C:\b\s\w\ir\tmp\t USERDOMAIN: WIN10-727F49A0- USERDOMAIN_ROAMINGPROFILE: WIN10-727F49A0- USERNAME: chrome-bot USERPROFILE: C:\Users\chrome-bot VPYTHON_VIRTUALENV_ROOT: C:\b\s\w\ir\cache\vpython WINDIR: C:\Windows Debugging info: CURRENT_DIR: 'C:\\b\\s\\w\\ir\\kitchen-workdir' DEPOT_TOOLS_DIR: 'C:\\b\\s\\w\\ir\\kitchen-checkout\\depot_tools' THIS_DIR: 'C:\\b\\s\\w\\ir\\kitchen-checkout\\depot_tools\\recipes\\recipe_modules\\bot_update\\resources' Gclient Solutions ================= boringssl (https://boringssl.googlesource.com/boringssl.git) Dependencies file is util/bot/DEPS Managed mode is OFF ===Running C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\gclient.py recurse -v git update-index --refresh === In directory: C:\b\s\w\ir\kitchen-workdir WARNING: Your metrics.cfg file was invalid or nonexistent. A new one will be created. Can't find .gclient_entries You need to run gclient sync at least once to use 'recurse'. This is because .gclient_entries needs to exist and be up to date. ===Failed in 0.0 mins of C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\gclient.py recurse -v git update-index --refresh === WARNING: Failed to update git indices. Revisions: ['boringssl@23e1a1f2d3c4ca0121eb93eb7d210501f9702317'] Fetching Git checkout at boringssl@23e1a1f2d3c4ca0121eb93eb7d210501f9702317 Using Python version: 2.7.13 (v2.7.13:a06454b1afa1, Dec 17 2016, 20:53:40) [MSC v.1500 64 bit (AMD64)] Checking git version... ===Running git.bat version === In directory: C:\b\s\w\ir\kitchen-workdir git version 2.17.1.windows.1 / Infra wrapper (infra/tools/git/windows-amd64 @ 7_rn53YpLArq1DleaKLY8QswA1RJBqsMFoj26VnTcY0C) ===Succeeded in 0.0 mins of git.bat version === Using git version 2.17.1.windows.1 / Infra wrapper (infra/tools/git/windows-amd64 @ 7_rn53YpLArq1DleaKLY8QswA1RJBqsMFoj26VnTcY0C) Fetching Git checkout ===Running C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\git_cache.py exists --quiet --cache-dir C:\\b\\s\\w\\ir\\cache\\git https://boringssl.googlesource.com/boringssl.git === In directory: C:\b\s\w\ir\kitchen-workdir C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl ===Succeeded in 0.0 mins of C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\git_cache.py exists --quiet --cache-dir C:\\b\\s\\w\\ir\\cache\\git https://boringssl.googlesource.com/boringssl.git === ===Running git.bat cat-file -e 23e1a1f2d3c4ca0121eb93eb7d210501f9702317 === In directory: C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl ===Failed in 0.0 mins of git.bat cat-file -e 23e1a1f2d3c4ca0121eb93eb7d210501f9702317 === ===Running C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\git_cache.py populate --ignore_locks -v --cache-dir C:\\b\\s\\w\\ir\\cache\\git https://boringssl.googlesource.com/boringssl.git --reset-fetch-config === In directory: C:\b\s\w\ir\kitchen-workdir C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl has 1 .pack files, re-bootstrapping if >50 running "git config --unset-all remote.origin.fetch" in "C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl" running "git config gc.autodetach 0" in "C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl" running "git config core.deltaBaseCacheLimit 2g" in "C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl" running "git config remote.origin.url https://boringssl.googlesource.com/boringssl.git" in "C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl" running "git config --replace-all remote.origin.fetch +refs/heads/*:refs/heads/* \+refs/heads/\*:.*" in "C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl" Fetching +refs/heads/*:refs/heads/* running "git fetch -v --progress origin +refs/heads/*:refs/heads/*" in "C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl" sh: warning: setlocale: LC_ALL: cannot change locale (English_United States.UTF-8): No such file or directory sh: warning: setlocale: LC_ALL: cannot change locale (English_United States.UTF-8): No such file or directory POST git-upload-pack (1008 bytes) remote: Counting objects: 1 remote: Counting objects: 36, done remote: Finding sources: 2% (1/39) remote: Finding sources: 5% (2/39) remote: Finding sources: 7% (3/39) remote: Finding sources: 10% (4/39) remote: Finding sources: 12% (5/39) remote: Finding sources: 15% (6/39) remote: Finding sources: 17% (7/39) remote: Finding sources: 20% (8/39) remote: Finding sources: 23% (9/39) remote: Finding sources: 25% (10/39) remote: Finding sources: 28% (11/39) remote: Finding sources: 30% (12/39) remote: Finding sources: 33% (13/39) remote: Finding sources: 35% (14/39) remote: Finding sources: 38% (15/39) remote: Finding sources: 41% (16/39) remote: Finding sources: 43% (17/39) remote: Finding sources: 46% (18/39) remote: Finding sources: 48% (19/39) remote: Finding sources: 51% (20/39) remote: Finding sources: 53% (21/39) remote: Finding sources: 56% (22/39) remote: Finding sources: 58% (23/39) remote: Finding sources: 61% (24/39) remote: Finding sources: 64% (25/39) remote: Finding sources: 66% (26/39) remote: Finding sources: 69% (27/39) remote: Finding sources: 71% (28/39) remote: Finding sources: 74% (29/39) remote: Finding sources: 76% (30/39) remote: Finding sources: 79% (31/39) remote: Finding sources: 82% (32/39) remote: Finding sources: 84% (33/39) remote: Finding sources: 87% (34/39) remote: Finding sources: 89% (35/39) remote: Finding sources: 92% (36/39) remote: Finding sources: 94% (37/39) remote: Finding sources: 97% (38/39) remote: Finding sources: 100% (39/39) remote: Finding sources: 100% (39/39) remote: Total 39 (delta 14), reused 35 (delta 14) From https://boringssl.googlesource.com/a/boringssl = [up to date] 2214 -> 2214 = [up to date] 2272 -> 2272 = [up to date] 2311 -> 2311 = [up to date] 2357 -> 2357 = [up to date] 2490 -> 2490 = [up to date] 2564 -> 2564 = [up to date] 2623 -> 2623 = [up to date] 2661 -> 2661 = [up to date] 2704 -> 2704 = [up to date] 2883 -> 2883 = [up to date] 2924 -> 2924 = [up to date] 2987 -> 2987 = [up to date] 3029 -> 3029 = [up to date] 3071 -> 3071 = [up to date] 3112 -> 3112 = [up to date] 3202 -> 3202 = [up to date] 3239 -> 3239 = [up to date] 3282 -> 3282 = [up to date] 3359 -> 3359 = [up to date] 3538 -> 3538 = [up to date] chromium-stable -> chromium-stable = [up to date] chromium-stable-with-bazel -> chromium-stable-with-bazel = [up to date] fips-20180730 -> fips-20180730 = [up to date] infra/config -> infra/config 3859fc883..23e1a1f2d master -> master bb8748bd3..3de99422c master-with-bazel -> master-with-bazel fetch +refs/heads/*:refs/heads/* took 0.0 minutes ===Succeeded in 0.0 mins of C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\git_cache.py populate --ignore_locks -v --cache-dir C:\\b\\s\\w\\ir\\cache\\git https://boringssl.googlesource.com/boringssl.git --reset-fetch-config === ===Running C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\git_cache.py exists --quiet --cache-dir C:\\b\\s\\w\\ir\\cache\\git https://boringssl.googlesource.com/boringssl.git === In directory: C:\b\s\w\ir\kitchen-workdir C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl ===Succeeded in 0.0 mins of C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\git_cache.py exists --quiet --cache-dir C:\\b\\s\\w\\ir\\cache\\git https://boringssl.googlesource.com/boringssl.git === ===Running git.bat cat-file -e 23e1a1f2d3c4ca0121eb93eb7d210501f9702317 === In directory: C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl ===Succeeded in 0.0 mins of git.bat cat-file -e 23e1a1f2d3c4ca0121eb93eb7d210501f9702317 === ===Running C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\git_cache.py exists --quiet --cache-dir C:\\b\\s\\w\\ir\\cache\\git https://boringssl.googlesource.com/boringssl.git === In directory: C:\b\s\w\ir\kitchen-workdir C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl ===Succeeded in 0.0 mins of C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\git_cache.py exists --quiet --cache-dir C:\\b\\s\\w\\ir\\cache\\git https://boringssl.googlesource.com/boringssl.git === ===Running git.bat clone --no-checkout --local --shared C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl C:\b\s\w\ir\kitchen-workdir\boringssl === In directory: C:\b\s\w\ir\kitchen-workdir Cloning into 'C:\b\s\w\ir\kitchen-workdir\boringssl'... sh: warning: setlocale: LC_ALL: cannot change locale (English_United States.UTF-8): No such file or directory done. ===Succeeded in 0.0 mins of git.bat clone --no-checkout --local --shared C:\\b\\s\\w\\ir\\cache\\git\boringssl.googlesource.com-boringssl C:\b\s\w\ir\kitchen-workdir\boringssl === ===Running git.bat config gc.auto 0 === In directory: C:\b\s\w\ir\kitchen-workdir\boringssl ===Succeeded in 0.0 mins of git.bat config gc.auto 0 === ===Running git.bat config gc.autodetach 0 === In directory: C:\b\s\w\ir\kitchen-workdir\boringssl ===Succeeded in 0.0 mins of git.bat config gc.autodetach 0 === ===Running git.bat config gc.autopacklimit 0 === In directory: C:\b\s\w\ir\kitchen-workdir\boringssl ===Succeeded in 0.0 mins of git.bat config gc.autopacklimit 0 === ===Running git.bat remote set-url --push origin https://boringssl.googlesource.com/boringssl.git === In directory: C:\b\s\w\ir\kitchen-workdir\boringssl ===Succeeded in 0.0 mins of git.bat remote set-url --push origin https://boringssl.googlesource.com/boringssl.git === ===Running git.bat checkout --force 23e1a1f2d3c4ca0121eb93eb7d210501f9702317 -- === In directory: C:\b\s\w\ir\kitchen-workdir\boringssl Note: checking out '23e1a1f2d3c4ca0121eb93eb7d210501f9702317'. You are in 'detached HEAD' state. You can look around, make experimental changes and commit them, and you can discard any commits you make in this state without impacting any branches by performing another checkout. If you want to create a new branch to retain commits you create, you may do so (now or later) by using -b with the checkout command again. Example: git checkout -b <new-branch-name> HEAD is now at 23e1a1f2d Test and fix an ABI issue with small parameters. ===Succeeded in 0.0 mins of git.bat checkout --force 23e1a1f2d3c4ca0121eb93eb7d210501f9702317 -- === ===Running git.bat clean -dff === In directory: C:\b\s\w\ir\kitchen-workdir\boringssl ===Succeeded in 0.0 mins of git.bat clean -dff === ===Running git.bat log --format=%H --max-count=1 === In directory: C:\b\s\w\ir\kitchen-workdir\boringssl 23e1a1f2d3c4ca0121eb93eb7d210501f9702317 ===Succeeded in 0.0 mins of git.bat log --format=%H --max-count=1 === ===Running git.bat config --global user.name === In directory: C:\b\s\w\ir\kitchen-workdir boringssl-ci-builder ===Succeeded in 0.0 mins of git.bat config --global user.name === ===Running git.bat config --global user.email === In directory: C:\b\s\w\ir\kitchen-workdir boringssl-ci-builder@chops-service-accounts.iam.gserviceaccount.com ===Succeeded in 0.0 mins of git.bat config --global user.email === ===Running C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\gclient.py sync --verbose --reset --force --ignore_locks --output-json c:\b\s\w\ir\tmp\t\tmpvfzhzn.json --nohooks --noprehooks --delete_unversioned_trees --break_repo_locks --revision boringssl@unmanaged === In directory: C:\b\s\w\ir\kitchen-workdir Looking for .gclient starting from C:\b\s\w\ir\kitchen-workdir Loaded .gclient config in C:\b\s\w\ir\kitchen-workdir: solutions = [ { 'deps_file': 'util/bot/DEPS', 'managed': False, 'name': 'boringssl', 'url': 'https://boringssl.googlesource.com/boringssl.git'}] cache_dir = r"C:\\b\\s\\w\\ir\\cache\\git" target_os_only=False gclient(743) ParseDepsFile:Found recursedeps "{'boringssl/util/bot/android_tools': 'util\\\\bot\\\\DEPS'}". boringssl (Elapsed: 0:00:00) ---------------------------------------- [0:00:00] Started. _____ boringssl at refs/remotes/origin/master [0:00:00] ________ unmanaged solution; skipping boringssl [0:00:00] Finished. ---------------------------------------- boringssl/util/bot/libFuzzer (Elapsed: 0:00:00) ---------------------------------------- [0:00:00] Started. _____ boringssl\util\bot\libFuzzer at 2a53098584c48af50aec3fb51febe5e651489774 [0:00:00] running "git cat-file -e 2a53098584c48af50aec3fb51febe5e651489774^^^^{commit}" in "C:\b\s\w\ir\cache\git\chromium.googlesource.com-chromium-llvm--project-compiler--rt-lib-fuzzer" skipping mirror update, it has rev=2a53098584c48af50aec3fb51febe5e651489774 already ________ running 'git -c core.deltaBaseCacheLimit=2g clone --no-checkout --progress --shared --verbose C:\b\s\w\ir\cache\git\chromium.googlesource.com-chromium-llvm--project-compiler--rt-lib-fuzzer C:\b\s\w\ir\kitchen-workdir\boringssl\util\bot\_gclient_libFuzzer_mfjh2y' in 'C:\b\s\w\ir\kitchen-workdir' [0:00:00] Cloning into 'C:\b\s\w\ir\kitchen-workdir\boringssl\util\bot\_gclient_libFuzzer_mfjh2y'... [0:00:00] done. ________ running 'git -c core.deltaBaseCacheLimit=2g fetch origin --prune --verbose' in 'C:\b\s\w\ir\kitchen-workdir\boringssl\util\bot\libFuzzer' [0:00:00] From C:\b\s\w\ir\cache\git\chromium.googlesource.com-chromium-llvm--project-compiler--rt-lib-fuzzer [0:00:00] = [up to date] master -> origin/master [0:00:01] Checked out 2a53098584c48af50aec3fb51febe5e651489774 to a detached HEAD. Before making any commits in this repo, you should use 'git checkout <branch>' to switch to an existing branch or use 'git checkout origin -b <branch>' to create a new branch for your work. [0:00:01] Finished. ---------------------------------------- boringssl/util/bot/gyp (Elapsed: 0:00:01) ---------------------------------------- [0:00:00] Started. _____ boringssl\util\bot\gyp at d61a9397e668fa9843c4aa7da9e79460fe590bfb [0:00:00] running "git cat-file -e d61a9397e668fa9843c4aa7da9e79460fe590bfb^^^^{commit}" in "C:\b\s\w\ir\cache\git\chromium.googlesource.com-external-gyp" skipping mirror update, it has rev=d61a9397e668fa9843c4aa7da9e79460fe590bfb already ________ running 'git -c core.deltaBaseCacheLimit=2g clone --no-checkout --progress --shared --verbose C:\b\s\w\ir\cache\git\chromium.googlesource.com-external-gyp C:\b\s\w\ir\kitchen-workdir\boringssl\util\bot\_gclient_gyp_ridijg' in 'C:\b\s\w\ir\kitchen-workdir' [0:00:00] Cloning into 'C:\b\s\w\ir\kitchen-workdir\boringssl\util\bot\_gclient_gyp_ridijg'... [0:00:00] done. ________ running 'git -c core.deltaBaseCacheLimit=2g fetch origin --prune --verbose' in 'C:\b\s\w\ir\kitchen-workdir\boringssl\util\bot\gyp' [0:00:00] From C:\b\s\w\ir\cache\git\chromium.googlesource.com-external-gyp [0:00:00] = [up to date] master -> origin/master [0:00:00] = [up to date] infra/config -> origin/infra/config [0:00:00] = [up to date] md-pages -> origin/md-pages [0:00:01] Checked out d61a9397e668fa9843c4aa7da9e79460fe590bfb to a detached HEAD. Before making any commits in this repo, you should use 'git checkout <branch>' to switch to an existing branch or use 'git checkout origin -b <branch>' to create a new branch for your work. [0:00:01] Finished. ---------------------------------------- ===Succeeded in 0.0 mins of C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\gclient.py sync --verbose --reset --force --ignore_locks --output-json c:\b\s\w\ir\tmp\t\tmpvfzhzn.json --nohooks --noprehooks --delete_unversioned_trees --break_repo_locks --revision boringssl@unmanaged === ===Running git.bat ls-files .DEPS.git === In directory: boringssl ===Succeeded in 0.0 mins of git.bat ls-files .DEPS.git === ===Running git.bat rev-parse HEAD === In directory: boringssl/ 23e1a1f2d3c4ca0121eb93eb7d210501f9702317 ===Succeeded in 0.0 mins of git.bat rev-parse HEAD === ===Running git.bat log --format=%B -n1 HEAD === In directory: boringssl/ Test and fix an ABI issue with small parameters. Calling conventions must specify how to handle arguments smaller than a machine word. Should the caller pad them up to a machine word size with predictable values (zero/sign-extended), or should the callee tolerate an arbitrary bit pattern? Annoyingly, I found no text in either SysV or Win64 ABI documentation describing any of this and resorted to experiment. The short answer is that callees must tolerate an arbitrary bit pattern on x86_64, which means we must test this. See the comment in abi_test::internal::ToWord for the long answer. CHECK_ABI now, if the type of the parameter is smaller than crypto_word_t, fills the remaining bytes with 0xaa. This is so the number is out of bounds for code expecting either zero or sign extension. (Not that crypto assembly has any business seeing negative numbers.) Doing so reveals a bug in ecp_nistz256_ord_sqr_mont. The rep parameter is typed int, but the code expected uint64_t. In practice, the compiler will always compile this correctly because: - On both Win64 and SysV, rep is a register parameter. - The rep parameter is always a constant, so the compiler has no reason to leave garbage in the upper half. However, I was indeed able to get a bug out of GCC via: uint64_t foo = (1ull << 63) | 2; // Some global the compiler can't // prove constant. ecp_nistz256_ord_sqr_mont(res, a, foo >> 1); Were ecp_nistz256_ord_sqr_mont a true int-taking function, this would act like ecp_nistz256_ord_sqr_mont(res, a, 1). Instead, it hung. Fix this by having it take a full-width word. This mess has several consequences: - ABI testing now ideally needs a functional testing component to fully cover this case. A bad input might merely produce the wrong answer. Still, this is fairly effective as it will cause most code to either segfault or loop forever. (Not the enc parameter to AES however...) - We cannot freely change the type of assembly function prototypes. If the prototype says int or unsigned, it must be ignoring the upper half and thus "fixing" it to size_t cannot have handled the full range. (Unless it was simply wrong of the parameter is already bounded.) If the prototype says size_t, switching to int or unsigned will hit this type of bug. The former is a safer failure mode though. - The simplest path out of this mess: new assembly code should *only* ever take word-sized parameters. This is not a tall order as the bad parameters are usually ints that should have been size_t. Calling conventions are hard. Change-Id: If8254aff8953844679fbce4bd3e345e5e2fa5213 Reviewed-on: https://boringssl-review.googlesource.com/c/34627 Commit-Queue: David Benjamin <davidben@google.com> Reviewed-by: Adam Langley <agl@google.com> ===Succeeded in 0.0 mins of git.bat log --format=%B -n1 HEAD === ===Running C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\gclient.py revinfo -a === In directory: C:\b\s\w\ir\kitchen-workdir boringssl: https://boringssl.googlesource.com/boringssl.git@23e1a1f2d3c4ca0121eb93eb7d210501f9702317 boringssl/util/bot/gyp: https://chromium.googlesource.com/external/gyp.git@d61a9397e668fa9843c4aa7da9e79460fe590bfb boringssl/util/bot/libFuzzer: https://chromium.googlesource.com/chromium/llvm-project/compiler-rt/lib/fuzzer.git@2a53098584c48af50aec3fb51febe5e651489774 ===Succeeded in 0.0 mins of C:\b\s\w\ir\cipd_bin_packages\bin\python.exe -u C:\b\s\w\ir\kitchen-checkout\depot_tools\gclient.py revinfo -a ===