Home | Search | fuchsia - Builders
Login

Builder go-x64-mac Build 8927864156707098944 Apple OSX

Results:

Success

Trigger Info:

Projectfuchsia
Revision550b8ac3263a232e48affe7d282fd2fd4f7bce29
Got Revision550b8ac3263a232e48affe7d282fd2fd4f7bce29

Execution:

Steps and Logfiles:

Show:
  1. ( 73 ms ) setup_build
    running recipe: "go_toolchain"
  2. ( 565 ms ) makedirs
  3. ( 39 ms ) git init
  4. ( 33 ms ) git remote
  5. ( 32 secs ) git fetch
  6. ( 3 secs ) git checkout
  7. ( 34 ms ) git rev-parse
  8. ( 72 ms ) git clean
  9. ( 217 ms ) submodule sync
  10. ( 216 ms ) submodule update
  11. ( 1 mins 5 secs ) build
  12. ( 544 ms ) read go version
  13. ( 1 mins 31 secs ) register fuchsia/go/mac-amd64
  14. ( 10 secs ) gsutil cp

Build Properties:

NameValueSource
$recipe_engine/path {"cache_dir": "/b/s/w/ir/cache", "temp_dir": "/b/s/w/ir/tmp/rt"} setup_build
$recipe_engine/runtime {"is_experimental": false, "is_luci": true} setup_build
bot_id "fuchsia-ci-mac-lab01-0003" setup_build
branch "refs/heads/master" setup_build
buildbucket {"build": {"bucket": "luci.fuchsia.ci", "created_by": "user:luci-scheduler@appspot.gserviceaccount.com", "created_ts": 1544122275592004, "id": "8927864156707098944", "project": "fuchsia", "tags": ["builder:go-x64-mac", "buildset:commit/git/550b8ac3263a232e48affe7d282fd2fd4f7bce29", "buildset:commit/gitiles/fuchsia.googlesource.com/third_party/go/+/550b8ac3263a232e48affe7d282fd2fd4f7bce29", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9093294218836579280", "scheduler_job_id:fuchsia/go-x64-mac", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"} setup_build
buildername "go-x64-mac" setup_build
gcs_bucket "fuchsia-build" setup_build
packages ["topaz/packages/buildbot"] setup_build
path_config "generic" setup_build
platform "mac-amd64" setup_build
project "integration" setup_build
recipe "go_toolchain" setup_build
remote "https://fuchsia.googlesource.com/integration" setup_build
repository "https://fuchsia.googlesource.com/third_party/go" setup_build
revision "550b8ac3263a232e48affe7d282fd2fd4f7bce29" setup_build
got_revision "550b8ac3263a232e48affe7d282fd2fd4f7bce29" git rev-parse

Blamelist:

  1. Marco Vanotti (mvanottiohnoyoudont@google.com)

Timing:

Create Thursday, 06-Dec-18 18:51:15 UTC
Start Thursday, 06-Dec-18 18:51:17 UTC
End Thursday, 06-Dec-18 18:55:54 UTC
Pending 2 secs
Execution 4 mins 36 secs

All Changes:

  1. [fidl][go] don't panic on unmarshalInline.

    Changed by Marco Vanotti - mvanottiohnoyoudont@google.com
    Changed at Thursday, 06-Dec-18 05:19:25 UTC
    Repository https://fuchsia.googlesource.com/third_party/go
    Branch
    Revision 550b8ac3263a232e48affe7d282fd2fd4f7bce29

    Comments

    [fidl][go] don't panic on unmarshalInline.
    
    This CL changes the way unmarshalInline behaves by making it
    `safeReadUint` instead of `readInt`.
    
    The issue with `readInt` is that it will panic if there's not enough
    data in the decoder buffer to read. So if you try to Unmarshall a
    message and provide a buffer that is not big enough, the service will
    just crash because of the panic.
    
    The CL refactors the logic so that instead of reading a hardcoded amount
    of data from the buffer, it will try to read the real size that the type
    has (given that these are basic data types, it should be OK and should
    match the real amount of bytes the underlying struct field occupies). If
    that read fails, it would just return an error.
    
    I have added some tests under fidl_test to make sure that the
    `Unmarshal` function doesn't panic if the payload is too small, and that
    the error returned is `ErrPayloadTooSmall`. These tests also test
    `UnmarshalNew`.
    
    TEST=`fx run-test go_fidl_test`
    
    NET-2031 #Done
    
    Change-Id: Ie05af389ed39149d3f3178862b9ef26e6c459b27
    

    Changed files

    • src/syscall/zx/fidl/encoding.go
    • src/syscall/zx/fidl/fidl_test/encoding_new_test.go
    • src/syscall/zx/fidl/fidl_test/encoding_test.go