PVS-StudioWarnings

PVS-Studio warning V517 for DDKSamples

Nov 17th, 2014
161
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
C++ 0.92 KB | None | 0 0
  1. HRESULT CSensor::HandleSetReportingAndPowerStates(....)
  2. {
  3.   ....
  4.   else if (SENSOR_POWER_STATE_LOW_POWER == ulCurrentPowerState)
  5.   {
  6.     Trace(TRACE_LEVEL_ERROR,
  7.       "%s Power State value is not correct = LOW_POWER, "
  8.       "hr = %!HRESULT!", m_SensorName, hr);
  9.   }
  10.   else if (SENSOR_POWER_STATE_LOW_POWER == ulCurrentPowerState)
  11.   {
  12.     Trace(TRACE_LEVEL_ERROR,
  13.       "%s Power State value is not correct = FULL_POWER, "
  14.       "hr = %!HRESULT!", m_SensorName, hr);
  15.   }
  16.   ....
  17. }
  18.  
  19. This suspicious code was found in DDKSamples project by PVS-Studio static code analyzer.
  20. Warning message is:
  21. V517 The use of 'if (A) {...} else if (A) {...}' pattern was detected. There is a probability of logical error presence. Check lines: 5641, 5645. sensor.cpp 5641
  22.  
  23. PVS-Studio is a static analyzer for detecting bugs in the source code of applications written in C, C++, C++11, C++/CX. Site: http://www.viva64.com/en/pvs-studio/
Advertisement
Add Comment
Please, Sign In to add comment