[Webkit-unassigned] [Bug 278436] New: [WebRTC] Ignore -Wthread-safety-reference-return warnings in libwebrtc headers

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=278436

            Bug ID: 278436
           Summary: [WebRTC] Ignore -Wthread-safety-reference-return
                    warnings in libwebrtc headers
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebRTC
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: ddkilzer at webkit.org
                CC: youennf at gmail.com
        Depends on: 274508

Ignore -Wthread-safety-reference-return warnings in libwebrtc headers.

This is follow-on work for Bug 274508: Disable thread-safety-reference-return warnings in libwebrtc.

Example warnings:

In file included from Source/WebCore/platform/graphics/cocoa/VP9UtilitiesCocoa.mm:32:
In file included from Source/WebCore/platform/mediastream/libwebrtc/LibWebRTCProvider.h:38:
In file included from WebKitBuild/Release/usr/local/include/webrtc/api/peer_connection_interface.h:135:
In file included from WebKitBuild/Release/usr/local/include/webrtc/p2p/base/port.h:41:
WebKitBuild/Release/usr/local/include/webrtc/p2p/base/connection.h:307:12: error: returning variable 'rtt_estimate_' by reference requires holding mutex 'network_thread_' [-Werror,-Wthread-safety-reference-return]
  307 |     return rtt_estimate_;
      |            ^
In file included from Source/WebCore/platform/graphics/cocoa/VP9UtilitiesCocoa.mm:32:
In file included from Source/WebCore/platform/mediastream/libwebrtc/LibWebRTCProvider.h:38:
In file included from WebKitBuild/Release/usr/local/include/webrtc/api/peer_connection_interface.h:135:
WebKitBuild/Release/usr/local/include/webrtc/p2p/base/port.h:253:48: error: returning variable 'password_' by reference requires holding mutex 'thread_' [-Werror,-Wthread-safety-reference-return]
  253 |   const std::string& password() const { return password_; }
      |                                                ^
2 errors generated.


In file included from WebKitBuild/Release/DerivedSources/WebCore/unified-sources/UnifiedSource363.cpp:2:
In file included from Source/WebCore/platform/mediastream/libwebrtc/LibWebRTCProvider.cpp:52:
WebKitBuild/Release/usr/local/include/webrtc/pc/peer_connection_factory.h:108:12: error: returning variable 'options_' by reference requires holding mutex 'signaling_thread()' [-Werror,-Wthread-safety-reference-return]
  108 |     return options_;
      |            ^
1 error generated.


In file included from WebKitBuild/Release/DerivedSources/WebCore/unified-sources/UnifiedSource65.cpp:8:
In file included from Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCStatsCollector.cpp:26:
In file included from Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCStatsCollector.h:37:
In file included from WebKitBuild/Release/usr/local/include/webrtc/pc/rtc_stats_collector.h:33:
In file included from WebKitBuild/Release/usr/local/include/webrtc/pc/peer_connection_internal.h:24:
In file included from WebKitBuild/Release/usr/local/include/webrtc/pc/jsep_transport_controller.h:45:
WebKitBuild/Release/usr/local/include/webrtc/p2p/base/p2p_transport_channel.h:194:12: error: returning variable 'ports_' by reference requires holding mutex 'network_thread_' [-Werror,-Wthread-safety-reference-return]
  194 |     return ports_;
      |            ^
WebKitBuild/Release/usr/local/include/webrtc/p2p/base/p2p_transport_channel.h:198:12: error: returning variable 'pruned_ports_' by reference requires holding mutex 'network_thread_' [-Werror,-Wthread-safety-reference-return]
  198 |     return pruned_ports_;
      |            ^
WebKitBuild/Release/usr/local/include/webrtc/p2p/base/p2p_transport_channel.h:235:12: error: returning variable 'remote_candidates_' by reference requires holding mutex 'network_thread_' [-Werror,-Wthread-safety-reference-return]
  235 |     return remote_candidates_;
      |            ^
In file included from WebKitBuild/Release/DerivedSources/WebCore/unified-sources/UnifiedSource65.cpp:8:
In file included from Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCStatsCollector.cpp:26:
In file included from Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCStatsCollector.h:37:
In file included from WebKitBuild/Release/usr/local/include/webrtc/pc/rtc_stats_collector.h:33:
In file included from WebKitBuild/Release/usr/local/include/webrtc/pc/peer_connection_internal.h:24:
In file included from WebKitBuild/Release/usr/local/include/webrtc/pc/jsep_transport_controller.h:54:
WebKitBuild/Release/usr/local/include/webrtc/pc/jsep_transport_collection.h:51:12: error: returning variable 'bundle_groups_' by reference requires holding mutex 'sequence_checker_' [-Werror,-Wthread-safety-reference-return]
   51 |     return bundle_groups_;
      |            ^
In file included from WebKitBuild/Release/DerivedSources/WebCore/unified-sources/UnifiedSource65.cpp:8:
In file included from Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCStatsCollector.cpp:26:
In file included from Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCStatsCollector.h:37:
In file included from WebKitBuild/Release/usr/local/include/webrtc/pc/rtc_stats_collector.h:33:
In file included from WebKitBuild/Release/usr/local/include/webrtc/pc/peer_connection_internal.h:27:
In file included from WebKitBuild/Release/usr/local/include/webrtc/pc/rtp_transmission_manager.h:36:
WebKitBuild/Release/usr/local/include/webrtc/pc/transceiver_list.h:142:12: error: returning variable 'transceiver_stable_states_by_transceivers_' by reference requires holding mutex 'sequence_checker_' exclusively [-Werror,-Wthread-safety-reference-return]
  142 |     return transceiver_stable_states_by_transceivers_;
      |            ^
5 errors generated.


Referenced Bugs:

https://bugs.webkit.org/show_bug.cgi?id=274508
[Bug 274508] Disable thread-safety-reference-return warnings in libwebrtc
-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20240821/a1682ef2/attachment.htm>