-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41026/
-----------------------------------------------------------
(Updated Dec. 18, 2015, 11:51 p.m.)
Review request for mesos and Joris Van Remoortere.
Changes
-------
review addressed.
Bugs: MESOS-4069
https://issues.apache.org/jira/browse/MESOS-4069
Repository: mesos
Description
-------
recv_callback could be called from libevents receive callback and Socket::recv
for the same buffer event and different requests. There is a check for buffer
length at Socket::recv but not at libevent's receive callback. This could lead
to the incoming request for Socket::recv being swapped out even though the
buffer length is zero. This change adds a check for buffer length before
swapping out the receive request object.
Diffs (updated)
-----
3rdparty/libprocess/3rdparty/Makefile.am
04d0fa324bf6eab98a51fc08298b9da2992d29d1
3rdparty/libprocess/3rdparty/stout/cmake/StoutTestsConfigure.cmake
a27cb98fa45cbd135ebfeca65e215fb3ff054739
3rdparty/libprocess/3rdparty/stout/include/Makefile.am
b2dea9b93adfa3ea0415a0b5c81a369dd29b6cfe
3rdparty/libprocess/3rdparty/stout/include/stout/error.hpp
cd1bad14919b9d566d5201201d8c3b4a335d5b9f
3rdparty/libprocess/3rdparty/stout/include/stout/errorbase.hpp
1e9db7e1cd2f411efb94893ba2c29bd0694e4ddc
3rdparty/libprocess/3rdparty/stout/include/stout/internal/windows/grp.hpp
1b74be6e8330392f77bf8c5ad50397a4fea82b60
3rdparty/libprocess/3rdparty/stout/include/stout/internal/windows/pwd.hpp
60990bb4c82e0510cd665f5b05f24daae754de2d
3rdparty/libprocess/3rdparty/stout/include/stout/ip.hpp
ffeb2d721149f600f552021c127694183f1aa837
3rdparty/libprocess/3rdparty/stout/include/stout/mac.hpp
09c0d4905ed548d9c82ab6e9083108a3abc0e2aa
3rdparty/libprocess/3rdparty/stout/include/stout/net.hpp
3c80910653ad2f3b663f19395f10214029f1a75e
3rdparty/libprocess/3rdparty/stout/include/stout/numify.hpp
26a637bec1193dd51437bd689c34fbe6d1935d89
3rdparty/libprocess/3rdparty/stout/include/stout/os.hpp
14fbca6d222bdfc0e8be301050b4ea1a8a6e7758
3rdparty/libprocess/3rdparty/stout/include/stout/os/freebsd.hpp
269e35f56044d250697fe2ffdb181f8bcae64d37
3rdparty/libprocess/3rdparty/stout/include/stout/os/mkdtemp.hpp
a3cc8cdcd52c5edd3e11ae4ae28b1275526f381d
3rdparty/libprocess/3rdparty/stout/include/stout/os/posix/bootid.hpp
7ea5e4631b14c48455af7c9feec98bcb4ae79d75
3rdparty/libprocess/3rdparty/stout/include/stout/os/posix/fork.hpp
11557e3938d23ae253dd4d4eade85600123e0b22
3rdparty/libprocess/3rdparty/stout/include/stout/os/posix/mkdtemp.hpp
bbfbb8a07e9d59f3a22e6917e7f247a797f45de8
3rdparty/libprocess/3rdparty/stout/include/stout/os/posix/process.hpp
PRE-CREATION
3rdparty/libprocess/3rdparty/stout/include/stout/os/posix/sendfile.hpp
293f82f6730551491504721e0af28e9537540db1
3rdparty/libprocess/3rdparty/stout/include/stout/os/posix/signals.hpp
f46f5910f34de534413d4c7df2681baee7c9d348
3rdparty/libprocess/3rdparty/stout/include/stout/os/process.hpp
a91a9d4c6aa6bf35c63cea531c34b537431fd9dc
3rdparty/libprocess/3rdparty/stout/include/stout/os/raw/environment.hpp
80cc45b24cfe6f6429d4d0ed4acdafba51c590eb
3rdparty/libprocess/3rdparty/stout/include/stout/os/strerror.hpp
f95ff598696ff4a33117745c9361d58d5cca24e0
3rdparty/libprocess/3rdparty/stout/include/stout/os/sysctl.hpp
8cfc77fc30d1bd17f7d3d3b2e8ca449f331aa9ca
3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/mkdtemp.hpp
2cb73bbb996775e3764ad852ccda5076b41aef41
3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/process.hpp
PRE-CREATION
3rdparty/libprocess/3rdparty/stout/include/stout/posix/os.hpp
4cf693fb7e8c6bb3ad1920ebe90d61f0adb5dc99
3rdparty/libprocess/3rdparty/stout/include/stout/try.hpp
c444c0118d39ee6a5da4618d7c62784464377280
3rdparty/libprocess/3rdparty/stout/include/stout/windows.hpp
d46e262e0fd1c2de36f3bf19d8bd693c23bf58cd
3rdparty/libprocess/3rdparty/stout/include/stout/windows/error.hpp
27c5d5df7db54bcbb8b110512e5398f12a34a28b
3rdparty/libprocess/3rdparty/stout/include/stout/windows/os.hpp
e738cdbf5846950c475c159fb9a770acc45159f5
3rdparty/libprocess/3rdparty/stout/tests/dynamiclibrary_tests.cpp
27626ae28db090f1a002239ff5c674b82e8fc9a8
3rdparty/libprocess/3rdparty/stout/tests/numify_tests.cpp
444377df00922df12d4b3ed25b4cfe9071cff5c3
3rdparty/libprocess/3rdparty/stout/tests/os/signals_tests.cpp
8b78beba81735c379be254a0ce20e50b822e69d9
3rdparty/libprocess/3rdparty/stout/tests/os_tests.cpp
e0a898d91e3521d4b228621a81412e1dd5ddf63d
3rdparty/libprocess/3rdparty/stout/tests/protobuf_tests.cpp
bf2a2b8a9f67c6a4cf66b156b9c14fae015a8af0
3rdparty/libprocess/3rdparty/stout/tests/strings_tests.cpp
7715fa4baa150f370bdd0fd5c2f98dc4c6f5fc55
3rdparty/libprocess/Makefile.am 82778bbd34cfcf54b4268e019d634eb64e506d55
3rdparty/libprocess/README.md c3f309a7dc1c94882c4cc97eeaf0736c2fca0ba5
3rdparty/libprocess/cmake/Process3rdpartyConfigure.cmake
d36fa2fbe903fb278e6c00b47bfa4b81cf8f4673
3rdparty/libprocess/cmake/ProcessTestsConfigure.cmake
3a2e0999722007475c023ade75719093e35cfc80
3rdparty/libprocess/configure.ac 134f8ef4395d059e2901bbb48826fbfc4e6e3124
3rdparty/libprocess/include/process/event.hpp
d7f3447d1ae25c470650f784f0a223e85da3e264
3rdparty/libprocess/include/process/future.hpp
817fca2ba5352a2c1cea1cb6cf6ecc49821215e4
3rdparty/libprocess/include/process/http.hpp
f0666f0fa48c4f3a98332d12066561a02a715236
3rdparty/libprocess/include/process/mime.hpp
9d0dd1def4cfc7a047059e96ec61f4904c1c000a
3rdparty/libprocess/include/process/owned.hpp
c1a04cc86c224340f388efd59a84073b71716629
3rdparty/libprocess/include/process/process.hpp
d3eb6986c3c3b4d67188f870028215104b3d732c
3rdparty/libprocess/src/authentication_router.hpp
c2516516d412234b8d885643c374cf3c380437f8
3rdparty/libprocess/src/config.hpp 1e0c2a5a61c3d1a7b50057f876f88a157a5e4ed8
3rdparty/libprocess/src/io.cpp 9530192e2c7afbfa4bd9aa2af2ff40f00ee505a9
3rdparty/libprocess/src/libevent.cpp 61d056b5ecd218948a7a81b8af15155fc1e824d9
3rdparty/libprocess/src/libevent_ssl_socket.cpp
55b91dd47bb5bd5e97147d0af91c7899fd42702c
3rdparty/libprocess/src/process.cpp cff635e74d22185de7ae767bc268ef4d56ad89f8
3rdparty/libprocess/src/tests/http_tests.cpp
19261502be220aaa40add7ce30a9b2b65d1d9fdc
3rdparty/libprocess/src/tests/main.cpp
78858a2b84a439d8f8a60ec8bcb6ac3a308087a6
3rdparty/libprocess/src/tests/process_tests.cpp
df388ed4397ed25cc574e4cc134d84a5888bfc86
3rdparty/libprocess/src/tests/subprocess_tests.cpp
8f0a31341eebb168f78bc74f8afbaef9ba98e847
CHANGELOG 388b8b91df6fce826413649b0a1480af3f78e918
bootstrap.bat 9ca1421c1d5aae67cfba05176007908422d85a5e
configure.ac 40d60a63cdba41d06305f09141f4d14d6e229d95
docs/allocation-module.md 1dc1abaf9e28abc01de728634593b42c7cfc8af5
docs/architecture.md d1b3c0e7ffab1b9e70fc51282a46cf97b7718a48
docs/authentication.md 5f13181e93df4dc15a62d5540eeb43c40f91344d
docs/authorization.md 1fc0e3f9a686480ffc4deb25e49f867318d1321a
docs/c++-style-guide.md 7f5e51948616036378b8993134ba6df09dad5d88
docs/clang-format.md 206bc2ccf42e5593f715c6ccf0cc42e5cbc62469
docs/configuration.md c75f56ce849f6960b3b6246bfa6949156a82eabb
docs/containerizer-internals.md f620c32cdd4dd7cf6f245fad4e50925531cb4a39
docs/effective-code-reviewing.md fdfafc7d11f8455b68395cd2a4df6dab75f07b22
docs/fetcher.md ec7372293491ed9ee4e20dda3c2def72a77a3f84
docs/getting-started.md a679093da97cb40499f0c9fd5c255d647fed062a
docs/high-availability.md 31aa66220617a3f8606b185ef247c11f00735227
docs/home.md 51c19bb9d0d74698fcdda6197d32ed8f4a57d7c9
docs/maintenance.md e6bfe0f655581a6a72de4579bd7e5753625c0e51
docs/mesos-documentation-guide.md PRE-CREATION
docs/mesos-provisioner.md fdb298c2a954e903317ef56abbcfe2470a2dfd23
docs/networking-for-mesos-managed-containers.md
3b3fde4ff4f152c374abd0f2cb4a4b4d1e699fc4
docs/oversubscription.md 0b1c20bf9bae9c179d82f5f611638faf91f91431
docs/reconciliation.md ea11905720a3cd60e88b18e64e5b882f5e250166
docs/release-guide.md 7c9ee140b394d61581f736f2413969d650f18fa4
docs/sandbox.md a2ad226b0ee7969622495f090579f15029a2a083
docs/scheduler-http-api.md 4223a316c9a8c6eb8b40e984d051066e6da1d56c
docs/testing-patterns.md cc150d6204992b551fe820d5aab54d6a6f60597d
include/mesos/authorizer/authorizer.hpp
aa9bb8b556f80a7c9c0fa3db95643efb49368dd5
include/mesos/authorizer/authorizer.proto
7f981e6d9cdd79a7d58b943bdc5abc81b355092f
include/mesos/mesos.proto 2431fdd6b84625c6140a2b3913736bffada4e7f6
include/mesos/quota/quota.proto 03e816dcd4dead8326731ac221df7354c0610fed
include/mesos/v1/mesos.proto 4aed0980b28dc1000aa2821f35303b736bc5bff8
site/source/assets/css/main.css 73ca8946091a638a359b1d769961879a283fa099
site/source/blog/2015-12-16-mesos-0-26-0-released.md
f4b466dd52e5cc76721ab1aea33b61f2f5fc2fa2
site/source/downloads.html.md 6812968aab6803bdd5796c5d7a52711f32b8fdf3
site/source/index.html.md e6dfd36937b35ecb363adc93476d112130c67e2b
src/CMakeLists.txt bdc45ae604c940dadc27ab6e8b8a3327bd00642b
src/Makefile.am e6d48dc16135b5d147d036e851422686eff7d5ef
src/authorizer/local/authorizer.hpp eb95c9fdb393f69501841da0e8a2342ceb3ab7e0
src/authorizer/local/authorizer.cpp 46e181a907bc5248a403baf9e02200fee7fc4ba3
src/cli/execute.cpp a2b610f32da3deb9d3df99c225d22a425e03cdba
src/cmake/MesosProtobuf.cmake 273d74df581d85b9ef08a533c6ad8e31a23f1417
src/common/http.hpp 4f4cbf6e53588b72204f9628dea5696c71eb66a5
src/common/http.cpp 5198650ba6dc1ea3dab7912e5ef6e375bf9acc96
src/common/resources.cpp b9d31aef8babee212374e352c57fadbff02167f3
src/examples/java/TestFramework.java cbcdeedb0da22f82e274d088eb15a4128fd920fe
src/examples/java/TestLog.java a35a37af040deddd7181b22739be516308e32ea2
src/exec/exec.cpp 83dbee9dd2d9a4e7ebf395c8070bc7f9f8412ef1
src/files/files.hpp 7b65a0a4fbc591fdf22b6f88d6c74034bd8ab599
src/hdfs/hdfs.hpp abdb9b9013abc5e7c2e0d94492b7c1586d32625e
src/hdfs/hdfs.cpp 51f016b049c6e947c4d27fbbbf79c53f9ec5a51e
src/health-check/main.cpp 0beaed575ec865d81e6e3d83d8a0c894613acba4
src/jvm/jvm.hpp d5023e4050d109ca97cedb05f85e1c24202ac3b0
src/launcher/executor.cpp 09e7de6f2136ed9ca13e4822aeabcba2f7c5837d
src/launcher/fetcher.cpp 0ff859846f31a31ff8605d54f8b68b3281f5440d
src/linux/ns.hpp 244a811b299c29b1dcd6652bd26e861e04df3f54
src/log/replica.cpp 45b77178fe93e26ae0ffde5ff0f02f36e5150bcb
src/logging/logging.cpp f7619b18fa4a78b20951edad892ca5c616bbed55
src/master/allocator/mesos/hierarchical.hpp
99c742906874c30c39c159e58a65277ade3c07fd
src/master/allocator/mesos/hierarchical.cpp
5da825a1d578a9ee40b4985378fddb3c5fb3b416
src/master/http.cpp 8dfd67f7d51b8395953d6beb77b2d71bc538eacd
src/master/master.hpp 7cb0e1692644e51271588abffa832e08c536b838
src/master/master.cpp 470b542729b01f41fc6a2e601a7a6c3d0c5353d5
src/master/quota.cpp df2cc5877160df101222756739f042895ceacc95
src/master/quota_handler.cpp 11167879b2480d9c8dd6398ca39c479089ec2272
src/master/validation.cpp 6a43bce5b7df6a9d939245c4726d060fa19eb305
src/messages/messages.proto b66bac14056407140bb3cf5aa3e67ac94df1a2f8
src/python/cli/src/mesos/cli.py f342992e0c15b1b1464bf3717634f07847aacdfe
src/slave/cmake/FindCurl.cmake df81fd538b06bf0c3e842a502bd5ef4083d897b5
src/slave/containerizer/fetcher.hpp bbdce88da6e41dbb88681bc9d604b00923033b3d
src/slave/containerizer/fetcher.cpp 4ac9149980cf7f013b318218a1b29369c5dcff17
src/slave/containerizer/mesos/containerizer.cpp
8242190dffa4d011ee2728a9f0a04d3857767b69
src/slave/containerizer/mesos/isolators/cgroups/perf_event.cpp
4d82c2b2f231c59cbb600869a0f2b716c1e55f5e
src/slave/containerizer/mesos/isolators/filesystem/linux.cpp
0df27d66ccd84e73b7fae068367c5dd035eaecb7
src/slave/containerizer/mesos/isolators/network/port_mapping.cpp
22a8428427b758bae4a0518356d7933c4110cd9f
src/slave/containerizer/mesos/provisioner/docker/message.proto
5c032701671b275d86c6d9276791a46df814396c
src/slave/containerizer/mesos/provisioner/docker/metadata_manager.cpp
bf85038f6a7e9940f9c410e7cf4ab50c2432673a
src/slave/containerizer/mesos/provisioner/docker/puller.hpp
d12b3d0ec9c1c7ce174451e4035e106eedcdcfd1
src/slave/containerizer/mesos/provisioner/docker/puller.cpp
7e830bc00b1533092735063b4b93039058806111
src/slave/http.cpp 66abdcb12d670bc4b77fa5b5f8ce35220f9aa698
src/slave/qos_controllers/load.hpp 098a6d0b2dfc54b5b95a261a780eea70a838c12d
src/slave/qos_controllers/load.cpp 52520d6220784423c09001b8bd59e090ebf787ec
src/slave/slave.hpp b7586ce42bfac9d9885a3eb8d82deb94680c236c
src/slave/slave.cpp ef869695ffeb2e6d9ef0a78ddb676b1b7cd19afe
src/slave/state.hpp 9cc9df2f8afb2ae848db467892dfb7f54973a6a3
src/slave/state.cpp b912c554fc9cb5593f541a7803422c22dc03dffd
src/tests/attributes_tests.cpp 3f3dde1301566f279c3fc7dc24e3ef67039a4c14
src/tests/authorization_tests.cpp dcf348acc75b143e3a854bde17e6042dc90599d5
src/tests/common/http_tests.cpp 0ea06341b092cd6ad278075b12dd970b84c84464
src/tests/containerizer/memory_pressure_tests.cpp
4a03af2c9c0643d964b1d76e2096341b59bf5dce
src/tests/containerizer/provisioner_docker_tests.cpp
b57ee8681e0861cf93f5dcf9368c004a57df1374
src/tests/executor_http_api_tests.cpp
8d86df5fd54bcfe998eb7dc4302509e7a00e9d84
src/tests/fetcher_cache_tests.cpp 1fb1e213d3c35479789688d1a3a49a3c6058b198
src/tests/fetcher_tests.cpp 1831d896ca8a52bec4adf87a67b6af845079796c
src/tests/group_tests.cpp 77349465e0163c8aa6bed6deefe3f98efb442f3d
src/tests/hdfs_tests.cpp 29f156028af57b83ee2aa299f3b1c89e96d15fd0
src/tests/health_check_tests.cpp 0fbccc373204d3b9431c614bdd6d046cc07e4566
src/tests/hierarchical_allocator_tests.cpp
e239b4746494fcc2b362a83afb634a2ce5e25f9b
src/tests/main.cpp 942488e57419ace8b7a821f53024aced0f43c7d9
src/tests/master_maintenance_tests.cpp
a14435c0e3f44b5a61e2f5a9debd20f4e447491f
src/tests/master_quota_tests.cpp 0473869783a714766ed26fff61d7f8c56342df74
src/tests/mesos.hpp 1c6acabab9d189a6d3cc8d63359053fd230377b8
src/tests/mesos.cpp 50ba8c448fcc4cee4d69074472c4e0e0c6cf7c0d
src/tests/oversubscription_tests.cpp 7a75fb38e0177e33cf0e7cb82b4b9ebf8f05fe0a
src/tests/persistent_volume_endpoints_tests.cpp
c0feedee393b8475fd27b0af9344d306a392893e
src/tests/persistent_volume_tests.cpp
01b3c13751a5558d5f06edb8f650c8644dc54486
src/tests/repair_tests.cpp 63ec889c4954c2c60d3466952551aa25b3284ddf
src/tests/reservation_endpoints_tests.cpp
b8edd6fafedd4c2221a8d19c1ebc71254071a8c7
src/tests/reservation_tests.cpp f429472e6b93a5d6d8fe6a5f7d5b94fc331f7295
src/tests/resources_tests.cpp e4a3435adc14f3b6b278b32348a6991543d5a320
src/tests/scheduler_http_api_tests.cpp
4d23a5a8368e0ed126469fa4a90a889b339ad004
src/tests/slave_tests.cpp 109d31c8661c6b91243852e7ee00ce9fa1effe4b
src/tests/values_tests.cpp a4eb68ad13407f471a07a9a923ed31c7890da9f7
src/v1/resources.cpp 0c0eee6b3b7ae3254d7dc7b06187855c9b873764
support/apply-review.sh 3a73492090d94b5a166c9a2cd78c77b616dc820d
support/apply-reviews.py b8a0b6df70d7b4bd13ebfd9e797df30479e89ab2
support/docker_build.sh 7882bd20e4b6ee28dc117086386de8faac995a3e
support/hooks/post-rewrite af907de7c6dd44fd6a7f34f52aaaed4460c44bc1
support/hooks/pre-commit bdc12af4c2e3f1c32c6b7763c791e41d7e9d72d4
support/site-docker/README.md 957117e1211f06b40c5fb621039dd90aff562a87
Diff: https://reviews.apache.org/r/41026/diff/
Testing
-------
make check
Thanks,
Jojy Varghese
