1277014 - Fix -Wstring-conversion warnings in security/manager/ssl/

Status: RESOLVED FIXED

(Core :: Security: PSM, defect)

Description

[Chris Peterson [:cpeterson]]

clang's -Wstring-conversion warning warns about the use of string literals as boolean expressions. This is not uncommon in "always fail" assertions, but is usually a bug in any other case. "Always fail" assertions are better expressed using MOZ_ASSERT_UNREACHABLE() or MOZ_CRASH().

security/manager/ssl/nsNSSComponent.cpp:1694:16 [-Wstring-conversion] implicit conversion turns string literal into bool: 'const char [31]' to 'bool'
security/manager/ssl/nsNSSIOLayer.cpp:1333:16 [-Wstring-conversion] implicit conversion turns string literal into bool: 'const char [22]' to 'bool'
security/manager/ssl/nsNSSIOLayer.cpp:1341:16 [-Wstring-conversion] implicit conversion turns string literal into bool: 'const char [22]' to 'bool'
security/manager/ssl/nsNSSIOLayer.cpp:1349:16 [-Wstring-conversion] implicit conversion turns string literal into bool: 'const char [22]' to 'bool'
security/manager/ssl/nsNSSIOLayer.cpp:1357:16 [-Wstring-conversion] implicit conversion turns string literal into bool: 'const char [22]' to 'bool'

Comment 1

[Chris Peterson [:cpeterson]]

Attachment: Fix -Wstring-conversion warnings in security/manager/ssl/

Comment 2

[Dana Keeler (she/her) (use needinfo) (:keeler for reviews)]

Comment on attachment 8758371 [details] [diff] [review]
Fix -Wstring-conversion warnings in security/manager/ssl/

Review of attachment 8758371 [details] [diff] [review]:
-----------------------------------------------------------------

great - thanks

::: security/manager/ssl/nsNSSIOLayer.cpp
@@ +1329,5 @@
>  
>  static int
>  _PSM_InvalidInt(void)
>  {
> +    MOZ_ASSERT_UNREACHABLE("I/O method is invalid");

nit: if you feel like it, you can convert these 4-space indented lines to 2 spaces.

Comment 3

[Chris Peterson [:cpeterson]]

(In reply to David Keeler [:keeler] (use needinfo?) from comment #2)
> nit: if you feel like it, you can convert these 4-space indented lines to 2
> spaces.

Thanks. I'll fix these lines' indentation.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2327aa152050
Fix -Wstring-conversion warnings in security/manager/ssl/. r=keeler

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2327aa152050