diff --git a/webrtc/examples/BUILD.gn b/webrtc/examples/BUILD.gn index d575c65b0..0098d6978 100644 --- a/webrtc/examples/BUILD.gn +++ b/webrtc/examples/BUILD.gn @@ -399,6 +399,10 @@ if (is_ios || (is_mac && target_cpu != "x86")) { cflags_objc = [ # Enabled for cflags_objc in build/config/compiler/BUILD.gn. "-Wno-objc-missing-property-synthesis", + + # Hide the warning for SecRandomCopyBytes(), until we update to upstream. + # https://bugs.chromium.org/p/webrtc/issues/detail?id=6396 + "-Wno-unused-result", ] } diff --git a/webrtc/examples/objc/AppRTCMobile/third_party/SocketRocket/SRWebSocket.m b/webrtc/examples/objc/AppRTCMobile/third_party/SocketRocket/SRWebSocket.m index b9148cd38..b4868a789 100644 --- a/webrtc/examples/objc/AppRTCMobile/third_party/SocketRocket/SRWebSocket.m +++ b/webrtc/examples/objc/AppRTCMobile/third_party/SocketRocket/SRWebSocket.m @@ -504,7 +504,7 @@ - (void)didConnect CFHTTPMessageSetHeaderFieldValue(request, CFSTR("Host"), (__bridge CFStringRef)(_url.port ? [NSString stringWithFormat:@"%@:%@", _url.host, _url.port] : _url.host)); NSMutableData *keyBytes = [[NSMutableData alloc] initWithLength:16]; - assert(!SecRandomCopyBytes(kSecRandomDefault, keyBytes.length, keyBytes.mutableBytes)); + SecRandomCopyBytes(kSecRandomDefault, keyBytes.length, keyBytes.mutableBytes); if ([keyBytes respondsToSelector:@selector(base64EncodedStringWithOptions:)]) { _secKey = [keyBytes base64EncodedStringWithOptions:0]; @@ -528,7 +528,7 @@ - (void)didConnect [_urlRequest.allHTTPHeaderFields enumerateKeysAndObjectsUsingBlock:^(id key, id obj, BOOL *stop) { CFHTTPMessageSetHeaderFieldValue(request, (__bridge CFStringRef)key, (__bridge CFStringRef)obj); }]; - + NSData *message = CFBridgingRelease(CFHTTPMessageCopySerializedMessage(request)); CFRelease(request); @@ -1361,7 +1361,7 @@ - (void)_sendFrameWithOpcode:(SROpCode)opcode data:(id)data; } } else { uint8_t *mask_key = frame_buffer + frame_buffer_size; - assert(!SecRandomCopyBytes(kSecRandomDefault, sizeof(uint32_t), (uint8_t *)mask_key)); + SecRandomCopyBytes(kSecRandomDefault, sizeof(uint32_t), (uint8_t *)mask_key); frame_buffer_size += sizeof(uint32_t); // TODO: could probably optimize this with SIMD