dedis / onet

64%

master: 66%

LAST BUILD ON BRANCH issue305

branch: issue305

CHANGE BRANCH

x

Reset

• issue305
• travis_24
• master
• config_30
• documentation_12
• travis_race
• protocol_19
• app_tesht_33
• same_origin_35
• race_41
• simulation_43
• localhost_simulation_45
• coveralls_cothority759
• simulation_error_48
• status_734
• registernewservce_cothority532
• names_cothority690
• crypto_delete_54
• network_main_722
• network_766
• network_767
• loadsave_cothority491
• network_768
• network_fix_rest
• network_fix_id
• data_available_66
• rename_config_cothority773
• onet_server_cothority777
• localhost_race_71
• conode_server_73
• network_proto_75
• fix_public_api
• debugging_cothority764
• fix_monitor_77
• debugging_2_cothority764
• fix_goreportcard
• v1.0
• merge_from_v1.0_88
• same_origin_86
• fix_graceful_17
• error-event
• router_rxtx_93
• updater_v1
• ctrl_c_97
• mininet_85
• service_load_save_102
• only-one-letter
• fix_106_crypto
• libtest_110
• complete_rw_114
• fix_96
• simulation_118
• v1.0_fix_unknown_message
• v1.0_merge
• mininet_125
• logf_127
• goback_dir_simul
• changelog_132
• improve-simul
• improve-simul-butthistimemuchbetter
• deterlab_nodes_140
• deterlab_nodes_v1_140
• nodes_104
• fix_log_error
• roottree_159
• v1_update_170330
• cleanup_99
• debugging_163
• race_165
• client_bandwidth
• monitor_multi_171
• sosp-omniledger
• realpath_181
• proto_files
• monitor_race_186
• typo
• v2
• register_messages_190
• messageProxyStore_loglvl
• log_race
• race_197
• skipchain_register_function
• libtest_dbgrun
• small
• clienterrorcode_split
• ipv6_919
• messageproxy
• infinite_loop_210
• backport_portscan_212
• prescript
• v1_prescript
• mininet_usability
• v1_mininet_usability
• v1_mininet_1604
• mininet_1604
• channel_block_219
• gh-189-input-validation
• use_mktemp
• configure_max_packet_size
• adding_stable_script
• cleaning_up
• schnorr_signature_hash
• libtest_search
• docs
• broadcast_233
• protocol_service_235
• kyber
• nary_tree_with_root_242
• el_to_ro
• merge_tree
• travis_stable
• v1.2
• regression_248
• regression_248_v1.2
• dispatch_error_237
• add_suite_to_app
• norand
• boltdb
• norandsuite
• context_counter_io_1.2
• libtest_suites
• context_counter_io
• remove_service_storage
• no64
• adding_more_buckets
• boltdb-documentation
• cleanup
• issue277
• issue277-v1
• keypair
• output
• handle_failure
• uuid
• flaky_protocolerror
• kv1
• winfix
• fix_int
• libtest_improvements
• p256
• uuidv1
• tls
• pause
• bcast_err_slice
• getupd_fail
• suite_deterlab
• register_convenience
• issue274
• issue301
• allow_non_fqdn
• cleanup_comments
• localtest_cleanup
• subset
• test_skipchains
• nocerr
• issue315
• hostport
• localhost
• suite_first
• config_path
• issue323
• rearrange_tree
• rearrange_tree_v1
• byzgen_poc
• issue292
• nil_dereference
• issue940
• issue300
• crux_modifications
• issue331
• status_revert
• issue280
• issue300bz
• check_error
• issue346
• issue230
• issue271
• issue1010
• prep_v2
• generate_259
• counterio
• stable_without_goveralls
• remove_stable_makefile
• remove_coveralls_copy
• racy_test
• improve_websocket_errors
• debug_level
• overlay_leak
• ignore_eof
• ignore_eof_byzgen
• print_ws_error_main
• log_aligning
• log_aligning_master
• v2_merge
• test_timeout
• help_testing
• help_testing_byzgen
• server_logging
• remove_race_bgpoc
• remove_race
• protect_protocol_storage
• protocol_protect_storage_master
• catch_print_errors
• catch_print_errors_master
• issue390
• issue397
• v2.0.2
• modinfo
• sha256_dbname
• modinfo2
• mininet_readme
• merge_byzgen_poc
• issue409
• status_log
• wslog
• pass_test
• mutex_localtest
• novgo
• adding_waitdone
• adding_wait
• wait_processes
• block_start
• mininet_cleanup
• documentation
• better_cleanup_report
• closeall_calming
• closeall_reduce
• navigation
• remove_bn256
• deterlab_bn256
• parse_time
• remove_global
• mininet_documentation_example
• closeall_panic
• issue1326
• closeall_test_fail
• log_indent
• close_simulations
• runwait_parse
• simul_error_reporting
• ignore_hash
• panic_resolved
• issue430
• issue446
• is_rotation
• runwait_err
• simulation_316
• issue446x
• libtest_doc
• add_db_versions
• test_closing_fixes
• issue468
• fix_protocol_creation
• mutex_services
• parallel_close
• nb
• treenode_closing
• tmout
• improved_shutdown
• streaming
• no_nb
• url
• rate_control_273
• fl_generation_cothority_1538
• cache_control_273
• issue485
• add_servers_later
• no_panic
• pending_messages_slowdown_482
• treenodeinstance_rx_481
• roster_concat
• stable1.10
• v2_preview
• treenil
• doc_macos_data
• port_and_print
• check_treeMarshal
• test_failure
• 32bits_mode_505
• dispatch_err
• service_public_key
• randseed
• sim
• v3.0.0-pre1
• mod
• novartime
• mod3
• refuse_fake_server
• issue501
• v3.0.0-pre2
• issue509
• nover
• v3.0.0-pre3
• v3.0.0-pre4
• sec
• warn_toml_wrong_1736
• kyber_pre4
• v3.0.0-pre5
• v3.0.0
• overlay_cache_524
• simulate_with_redirect
• v3.0.1
• roster_id_fix
• v3.0.2
• service_aggregate
• v3.0.3
• register_all_bn
• v3.0.4
• ineiti-patch-1
• fix_treestorage_closed
• websock_url
• v3.0.5
• data_race_1828
• v3.0.6
• simul_measure_msg
• v3.0.7
• loadcothority
• v3.0.8
• private_url
• v3.0.9
• infinite_loop_1851
• v3.0.10
• stat_bucket_543
• darwin
• v3.0.11
• rest_json
• timeout-http-get
• fix_simulation_bandwidth
• v3.0.12
• v3.0.13
• json_error
• v3.0.14
• simulation_server_measure_550
• processor_panic_1934
• streaming_handler_close_554
• add_user_interesting_gr
• v3.0.15
• dial_timeout
• v3.0.16
• fix_panic_websocket
• v3.0.17
• sendprotobufparallel
• sendprotobufparallel-1
• v3.0.18
• v3.0.19
• v3.0.20
• sim_cleanup_error_567
• v3.0.21
• send_parallel_decoder
• leak_send_parallel
• cgrigis-patch-1
• v3.0.22
• v3.0.23
• reload_ssl_certificate
• bump_protobuf
• v3.0.24
• find_tree_issue_2064
• improve_error_handling
• leaks
• fix_travis_coding
• issue583
• v3.0.26
• v4_initial_commit
• v4.0.0-pre1
• crypto_interfaces_579
• doc-merge
• ciphersuite_579
• wait_some
• simulation-processor-registration-fix
• remove_log_errfatal
• 4to3
• v3.0.28
• v3.0.29
• use_goroutine_dispatcher
• issue604
• v3.0.30
• simulation_blocking_fix
• v3.0.31
• config_error_simple
• streaming-from-client
• v3.0.32
• errorPlusV
• v3.1.0
• fix_cey_issue
• v3.1.1
• tracing
• tracing_nounfold
• v3.0.33
• v3.2.0
• issue_627
• go114
• v3.2.1
• issue_631
• gofmt_fixes
• trim_honeycomb_size
• log_zero
• tracing_defaults
• add_nodename
• getid_id
• less_honeycomb
• v3.2.2
• equal_testing
• v3.2.3
• cothority_ns_link
• minor
• simul_docu
• update_mininet
• v3.2.4
• error_return
• go15
• v3.2.5
• check_new_tls_handshake
• gopkg.in
• google-uuid
• issue657
• issue601
• no_send_restrictions
• v3.2.6
• ubuntu
• log_defaults
• proxy-changes
• v3.2.7
• init_serveridentities
• v3.2.8
• websocket-update
• v3.2.9
• timeouts
• dependabot/go_modules/github.com/gorilla/websocket-1.4.1
• v3.2.10
• test_go_1_17

Build # 1376

Build Type

push

travis-ci

Committed by jeffallen

Commit Message

Remove race condition in message dispatcher

Code of the form "if len(chan)==0 { send }" is racy because two
goroutines could observe len == 0, and then both try to send. One
will block when it expected to not do so. Replace with a select
to correctly implement the desired behavior that the two goroutines
end up not blocked, and the channel ends up with one value on it.

We have not seen any indication this is causing a bug, but
leaving racy code around is asking for trouble.

Also: Make a flaky test stop failing, since the thing it is
checking is not very important. When this test runs against the
"local" router, it never fails, indicating that this is
a benign race in the TCP implementation, not in our Router.

Fixes #305.

Run Details

4525 of 7117 relevant lines covered (63.58%)

67.77 hits per line