[WTF] Coverity scan issues
authortpopela@redhat.com <tpopela@redhat.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 26 Jul 2018 09:00:09 +0000 (09:00 +0000)
committertpopela@redhat.com <tpopela@redhat.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 26 Jul 2018 09:00:09 +0000 (09:00 +0000)
https://bugs.webkit.org/show_bug.cgi?id=186800

Reviewed by Brent Fulgham.

* wtf/CheckedArithmetic.h: Swap the operands to multiply().
* wtf/ThreadSpecific.h: Initialize the m_key variable.
* wtf/dtoa.cpp: Initialize the next variable.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@234251 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WTF/ChangeLog
Source/WTF/wtf/CheckedArithmetic.h
Source/WTF/wtf/ThreadSpecific.h
Source/WTF/wtf/dtoa.cpp

index 52ada99..55e64b8 100644 (file)
@@ -1,3 +1,14 @@
+2018-07-26  Tomas Popela  <tpopela@redhat.com>
+
+        [WTF] Coverity scan issues
+        https://bugs.webkit.org/show_bug.cgi?id=186800
+
+        Reviewed by Brent Fulgham.
+
+        * wtf/CheckedArithmetic.h: Swap the operands to multiply().
+        * wtf/ThreadSpecific.h: Initialize the m_key variable.
+        * wtf/dtoa.cpp: Initialize the next variable.
+
 2018-07-25  Tomas Popela  <tpopela@redhat.com>
 
         Correctly close the variable argument list
index 31be98b..e668bf8 100644 (file)
@@ -489,7 +489,7 @@ template <typename ResultType> struct ArithmeticOperations<unsigned, int, Result
 
     static inline bool multiply(int64_t lhs, int64_t rhs, ResultType& result)
     {
-        return ArithmeticOperations<int, unsigned, ResultType>::multiply(rhs, lhs, result);
+        return ArithmeticOperations<int, unsigned, ResultType>::multiply(lhs, rhs, result);
     }
 
     static inline bool equals(unsigned lhs, int rhs)
index 1be1204..fc75b24 100644 (file)
@@ -122,7 +122,7 @@ private:
     void static THREAD_SPECIFIC_CALL destroy(void* ptr);
 
 #if USE(PTHREADS)
-    pthread_key_t m_key;
+    pthread_key_t m_key { };
 #elif OS(WINDOWS)
     int m_index;
 #endif
index edefd7f..7b5f592 100644 (file)
@@ -360,7 +360,7 @@ struct P5Node {
 public:
     P5Node() { }
     BigInt val;
-    P5Node* next;
+    P5Node* next { nullptr };
 };
 
 static P5Node* p5s;