aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/core
Commit message (Collapse)AuthorAge
* Merge pull request #13805 from AspirinSJL/lb_bugGravatar Juanli Shen2018-01-08
|\ | | | | Change client_load_report_timer_pending var name
| * Unify the timer callback boolsGravatar Juanli Shen2018-01-08
| |
* | Merge pull request #13922 from y-zeng/openssl_1_1_0Gravatar Yuchen Zeng2018-01-08
|\ \ | | | | | | Silence openssl 1.1.0 warnings
* \ \ Merge pull request #13933 from kpayson64/avoid_stallGravatar kpayson642018-01-08
|\ \ \ | | | | | | | | Avoid stall
| * | | Fix bug with pollhup workaroundGravatar Ken Payson2018-01-05
| | | |
* | | | Merge pull request #13931 from apolcyn/merge_18xGravatar apolcyn2018-01-05
|\ \ \ \ | | | | | | | | | | Merge 1.8.x into master
| | * | | Add more control flow loggingGravatar Ken Payson2018-01-05
| | | | |
| * | | | Manual merge conflict resolve: remove exec_ctx param from tcp_freeGravatar Alexander Polcyn2018-01-05
| | | | |
| | * | | Fix a Python spinlock bugGravatar Ken Payson2018-01-05
| | | | |
* | | | | Merge pull request #13900 from vjpai/odrGravatar Vijay Pai2018-01-05
|\ \ \ \ \ | | | | | | | | | | | | Wrap duplicated-name definitions in anonymous namespace (except for iomgr)
| | * | | | Merge remote-tracking branch 'upstream/v1.8.x' into merge_18xGravatar Alexander Polcyn2018-01-05
| |/| | | | |/| | | | |
| | | | * | Silence openssl 1.1.0 warningsGravatar Yuchen Zeng2018-01-04
| | | | | |
* | | | | | Set error status correctly on server sideGravatar Ken Payson2018-01-04
| |_|_|/ / |/| | | |
| | * | | Mark tcp errors as UNAVAILABLE in UV tcp codeGravatar murgatroid992018-01-04
| | | | |
| * | | | Revert unneeded changesGravatar Vijay Pai2018-01-03
| | | | |
* | | | | Merge pull request #13879 from AspirinSJL/client_load_report_bugGravatar Juanli Shen2018-01-03
|\ \ \ \ \ | | | | | | | | | | | | Restore checking initial request sent
| | * | | | Wrap duplicated-name definitions in anonymous namespaceGravatar Vijay Pai2018-01-02
| | | | | |
* | | | | | Revert "Fix a Python spinlock bug"Gravatar Mehrdad Afshari2018-01-02
| |/ / / / |/| | | | | | | | | | | | | | This reverts commit 2df509fc0e8628d6d4431139ce953c70796a21eb.
* | | | | Fix struct namesGravatar Vijay Pai2018-01-01
| | | | |
* | | | | Merge branch v1.8.x into masterGravatar Mehrdad Afshari2017-12-31
|\ \ \ \ \ | | |/ / / | |/| | | | | | | | Upmerge v1.8.3 into master
| * | | | Relax log-level when polling engines are undefinedGravatar Mehrdad Afshari2017-12-29
| | | | |
| | * | | Restore checking initial request sentGravatar Juanli Shen2017-12-27
| | | | |
* | | | | Merge pull request #13844 from danzh2010/startcbGravatar Yang Gao2017-12-22
|\ \ \ \ \ | | | | | | | | | | | | Change grpc_udp_server interface
* | | | | | Separate client and server keepalive_permit_without_callsGravatar Yuchen Zeng2017-12-21
| | | | | |
| * | | | | Merge branch 'master' into startcbGravatar Dan Zhang2017-12-20
| |\ \ \ \ \ | |/ / / / / |/| | | | |
* | | | | | Merge pull request #13834 from dgquintas/debug_location_namespaceGravatar David G. Quintas2017-12-20
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add namespace to macro expansion
| | * | | | | Add a start_cb to grpc_udp_listener to be called when listener isGravatar Dan Zhang2017-12-20
| |/ / / / / |/| | | | | | | | | | | | | | | | | created.
| * | | | | Add namespace to macro expansionGravatar David Garcia Quintas2017-12-20
| | | | | |
* | | | | | Merge pull request #13811 from ita9naiwa/masterGravatar Noah Eisen2017-12-19
|\ \ \ \ \ \ | | | | | | | | | | | | | | Typo fix
* \ \ \ \ \ \ Merge pull request #13831 from dgquintas/fix_wrong_slice_unrefGravatar David G. Quintas2017-12-19
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix wrong unreffing of slice
* \ \ \ \ \ \ \ Merge pull request #13833 from ncteisen/commentsGravatar Noah Eisen2017-12-19
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | | Add ownership semantic comments to error.h
| | | | * | | | Merge pull request #13665 from kpayson64/poll_spin_bugGravatar kpayson642017-12-19
| | | | |\ \ \ \ | | | | | | | | | | | | | | | | | | Fix a Python spinlock bug
| * | | | | | | | Add ownership semantic comments to error.hGravatar ncteisen2017-12-19
| | | | | | | | |
| | * | | | | | | Fix wrong unreffing of sliceGravatar David Garcia Quintas2017-12-19
| |/ / / / / / / |/| | | | | | |
| | | | * | | | Fix a Python spinlock bugGravatar Ken Payson2017-12-19
| | | | | | | |
* | | | | | | | Merge pull request #13494 from dgquintas/backoff_cppGravatar David G. Quintas2017-12-19
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | C++-ize backoff
| * | | | | | | | Create copy of "ac" data instead of stretching the locked sectionGravatar David Garcia Quintas2017-12-19
| | | | | | | | |
| * | | | | | | | Fix use-after-free caused by unsync'd access in tcp_client_posix.Gravatar David Garcia Quintas2017-12-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tc_on_alarm() and on_writable() race, resulting in the following: ``` D1219 08:59:33.425951347 86323 tcp_client_posix.cc:143] CLIENT_CONNECT: ipv4:127.0.0.1:27465: on_writable: error="No Error" D1219 08:59:33.426032150 86342 tcp_client_posix.cc:104] CLIENT_CONNECT: ipv4:127.0.0.1:27465: on_alarm: error="No Error" // At this point, note that the callbacks are running on different threads. D1219 08:59:33.426063521 86323 tcp_client_posix.cc:218] XXX on_writable ac->addr_str 0x603000008dd0 before unlock. # refs 2->1. Done 0 // on_writable() unrefs while still holding the lock. Because refs > 0, it marks its "done" as false and unlocks. D1219 08:59:33.426125130 86342 tcp_client_posix.cc:113] XXX tc_on_alarm ac->addr_str 0x603000008dd0 before unlock. # refs 1->0. Done 1 // right after on_writable() unlocks, tc_on_alarm() acquires the lock and unrefs, this time getting to zero and marking its "done" as true. // It then proceeds to destroy "ac", and, in particular for this failure, "ac->addr_str". D1219 08:59:33.426139370 86323 tcp_client_posix.cc:234] XXX on_writable about to read from ac->addr_str 0x603000008dd0. Done 0, error=OS Error // When on_writable() tries to read ac->addr_str to assemble its error details, it causes a use-after-free. ``` The problem is the lock isn't held long enough by on_writable(). Alternatively, a copy of ac->addr_str could be made in on_writable() while still holding the lock, but that seems more fragile. It doesn't seem that holding the lock longer would be a performance issue, given we are in a failure scenario.
* | | | | | | | | Fix internal TSAN bugGravatar Noah Eisen2017-12-19
| |/ / / / / / / |/| | | | | | |
* | | | | | | | Merge pull request #13798 from apolcyn/min_max_rpc_version_constantsGravatar apolcyn2017-12-18
|\ \ \ \ \ \ \ \ | |_|_|_|_|/ / / |/| | | | | | | Define RPC protocol version constants
| * | | | | | | Define RPC protocol version constantsGravatar Alexander Polcyn2017-12-18
| | |_|_|_|_|/ | |/| | | | |
| | | * | | | minor typo : becuase -> becauseGravatar ita9naiwa2017-12-19
| |_|/ / / / |/| | | | |
* | | | | | Merge pull request #13776 from ncteisen/executor-tsanGravatar Noah Eisen2017-12-15
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix shutdown executor tsan bug
* \ \ \ \ \ \ Merge pull request #13771 from yang-g/deleteGravatar Yang Gao2017-12-15
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Mark tcp errors as UNAVAILABLE
| | | | * \ \ \ Merge branch 'master' of github.com:grpc/grpc into backoff_cppGravatar David Garcia Quintas2017-12-15
| | | | |\ \ \ \ | |_|_|_|/ / / / |/| | | | | | |
| | * | | | | | Fix executor tsan bugGravatar ncteisen2017-12-14
| |/ / / / / / |/| | | | | |
* | | | | | | Fix tsan in bad_client testsGravatar Noah Eisen2017-12-13
| |/ / / / / |/| | | | |
| * | | | | Mark tcp errors as UNAVAILABLEGravatar yang-g2017-12-13
| | |_|_|/ | |/| | |
* | | | | Merge pull request #13754 from muxi/fix-stream-compression-gzipGravatar Muxi Yan2017-12-13
|\ \ \ \ \ | | | | | | | | | | | | Fix stream_compression_payload flake
| | | * \ \ Merge branch 'master' of github.com:grpc/grpc into backoff_cppGravatar David Garcia Quintas2017-12-13
| | | |\ \ \ | |_|_|/ / / |/| | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-08 22:13:02 +0000