Examples of errors detected by the V530 diagnostic

V530. Return value of 'Foo' function is required to be used.

TortoiseSVN

V530 The return value of function 'empty' is required to be utilized. contextmenu.cpp 434


STDMETHODIMP CShellExt::Initialize(....)
{
  ....
  ignoredprops.empty();
  for (int p=0; p<props.GetCount(); ++p)
  {
    if (props.GetItemName(p).
          compare(SVN_PROP_IGNORE)==0)
    {
      std::string st = props.GetItemValue(p);
      ignoredprops = UTF8ToWide(st.c_str());
      // remove all escape chars ('\\')
      std::remove(ignoredprops.begin(),
                  ignoredprops.end(), '\\');
      break;
    }
  }
  ....
}

The std::remove function does not remove elements from the container. It only shifts the elements and brings the iterator back to the beginning of the trash. Suppose we have the vector<int> container that contains elements 1,2,3,1,2,3,1,2,3. If we execute the code "remove( v.begin(), v.end(), 2 )", the container will contain elements 1,3,1,3,?,?,?, where ? is some trash. The function will bring the iterator back to the first senseless element, so if we want to remove these trash elements, we must write the code this way: "v.erase(remove(v.begin(), v.end(), 2), v.end())".

Similar errors can be found in some other places:

V530 The return value of function 'remove' is required to be utilized. contextmenu.cpp 442

TortoiseSVN

V530 The return value of function 'empty' is required to be utilized. mailmsg.cpp 40


CMailMsg& CMailMsg::SetFrom(string sAddress,
                            string sName)
{
   if (initIfNeeded())
   {
      // only one sender allowed
      if (m_from.size())
         m_from.empty();
      m_from.push_back(TStrStrPair(sAddress,sName));
   }
   return *this;
}

This is what should have been written here: m_from.clear();

TortoiseSVN

V530 The return value of function 'empty' is required to be utilized. mailmsg.cpp 54


CMailMsg& CMailMsg::SetTo(string sAddress,
                          string sName)
{
   if (initIfNeeded())
   {
      // only one recipient allowed
      if (m_to.size())
         m_to.empty();

      m_to.push_back(TStrStrPair(sAddress,sName));
   }
   return *this;
}

This is what should have been written here: m_to.clear();

WinMerge

V530 The return value of function 'empty' is required to be utilized. diractions.cpp 1312


/**
* @brief Get the file names on both sides for specified item.
* @note Return empty strings if item is special item.
*/
void CDirView::GetItemFileNames(int sel,
  String& strLeft, String& strRight) const
{
  UINT_PTR diffpos = GetItemKey(sel);
  if (diffpos == (UINT_PTR)SPECIAL_ITEM_POS)
  {
    strLeft.empty();
    strRight.empty();
  }
  else
  {
     ....
  }
}

This is what should have been written here: strLeft.clear(); strRight.clear();

WinMerge

V530 The return value of function 'empty' is required to be utilized. varprop.cpp 133


/**
* @brief Clear variant's value (reset to defaults).
*/
void VariantValue::Clear()
{
  m_vtype = VT_NULL;
  m_bvalue = false;
  m_ivalue = 0;
  m_fvalue = 0;
  m_svalue.empty();
  m_tvalue = 0;
}

This is what should have been written here: m_svalue.clear();

EchoVNC

V530 The return value of function 'empty' is required to be utilized. miniWinVNC parsecmdline.cpp 367


bool CParseCmdLine::CheckParameter(..., std::string& rStrForSave)
{
  ....
  rStrForSave.empty();
  ....
}

This is what should have been written here: rStrForSave.clear();

EchoVNC

V530 The return value of function 'empty' is required to be utilized. AskParameter wndproc.cpp 163


struct ST_DATA
{
  ....
  std::string m_szImagePath;
  ....
};

void Install_and_Run_MainFiles(const char* szPath)
{
  ....
  if (!WriteTmpBanner(ser, szPath))
  {
    ser.m_data.m_szImagePath.empty();
  }
  ....
}

This is what should have been written here: ser.m_data.m_szImagePath.clear();

IPP Samples

V530 The return value of function 'remove' is required to be utilized. h264_dec umc_h264_thread.cpp 226


void H264ThreadGroup::RemoveThread(H264Thread * thread)
{
    AutomaticUMCMutex guard(m_mGuard);
    std::remove(m_threads.begin(), m_threads.end(), thread);
}

Nmap Security Scanner

V530 The return value of function 'empty' is required to be utilized. nping nepcontext.cc 562


/** Deletes all previous field specifiers.
  * This should be used when dealing
  * with clients that send multiple NEP_PACKET_SPEC messages,
  * so only the last PacketSpec is taken into account. */
int NEPContext::resetClientFieldSpecs(){
  this->fspecs.empty();
  return OP_SUCCESS;
} /* End of resetClientFieldSpecs() */

What a nice comment. It's a pity that the code does quite a different thing. This is what should have been written here: this->fspecs.clear();

Chromium

V530 The return value of function 'empty' is required to be utilized. chrome_frame_ie protocol_sink_wrap.cc 399


std::wstring url_;

HRESULT ProtData::ReportProgress(IInternetProtocolSink* delegate,
                                 ULONG status_code,
                                 LPCWSTR status_text)
{
  ....
  case BINDSTATUS_REDIRECTING:
    url_.empty();
    if (status_text)
      url_ = status_text;
    break;
  ....
}

This is what should have been written here: url_.clear();

Chromium

V530 The return value of function 'empty' is required to be utilized. chrome_frame_npapi np_proxy_service.cc 293


bool NpProxyService::GetProxyValueJSONString(std::string* output)
{
  DCHECK(output);
  output->empty();
  ....
}

This is what should have been written here: output->clear();

Intel AMT SDK

V530 The return value of function 'empty' is required to be utilized. AMTredirection mcsol.cpp 443


bool MCSOL::_createHTLink()
{
  ....
  string HTcmd = TERM_CMD;
  ....
  HTcmd.empty();
  ....
}

This is what should have been written here: HTcmd.clear();

Intel AMT SDK

V530 The return value of function 'empty' is required to be utilized. EventLogReader cimclass.h 153


void CimClassContainer::Clear()
{
  for(unsigned int i = 0; i < vec.size(); i++)
  {
    delete vec[i];
    vec[i] = NULL;
  }
  vec.empty();
}

This is what should have been written here: vec.clear();

ReactOS

V530 The return value of function 'wcscmp' is required to be utilized. msdmo dmoreg.c 617


  if (ERROR_SUCCESS == hres)
  {
    Names[count] = HeapAlloc(GetProcessHeap(), 0,
                             strlenW(szValue) + 1);
    if (Names[count])
       strcmpW(Names[count], szValue);
  }

It's strange that the result of calling the strcmpW() function is not used in any way.

Battle for Wesnoth

V530 The return value of function 'back' is required to be utilized. wesnoth menu_events.cpp 91


gui::dialog_button_action::RESULT
delete_recall_unit::button_pressed(int menu_selection)
{
  const std::vector<std::pair<int, int> >& param =
    std::vector<std::pair<int, int> >();
  param.back();
  const size_t index = size_t(filter_.get_index(menu_selection));
  ....
}

It's strange that the result of calling the param.back() function is not used in any way.

wxWidgets

V530 The return value of function 'empty' is required to be utilized. base translation.cpp 989


bool wxMsgCatalogFile::LoadData(....)
{
  ....
  wxString m_charset;
  ....
  if ( m_charset == wxS("CHARSET") )
  {
    m_charset.empty();
  }
  ....
}

This is what should have been written here: m_charset.clear();

wxWidgets

V530 The return value of function 'empty' is required to be utilized. base registry.cpp 329


wxString m_strKey;

void wxRegKey::SetHkey(WXHKEY hKey)
{
  ....

  // reset old data
  m_strKey.empty();
  m_dwLastError = 0;
}

This is what should have been written here: m_strKey.clear();

Visualization Toolkit (VTK)

V530 The return value of function 'empty' is required to be utilized. vtkRendering vtklabelhierarchy.cxx 425


std::vector<int> Path;

void vtkLabelHierarchyFrustumIterator::Next()
{
  ....
  this->Path.empty();
  ....
}

This is what should have been written here: this->Path.clear();

MongoDB

V530 The return value of function 'remove' is required to be utilized. cmdline.cpp 106


void CmdLine::parseConfigFile( istream &f, stringstream &ss ) {
  ....
  f.getline(line, MAX_LINE_LENGTH);
  s = line;
  std::remove(s.begin(), s.end(), ' ');
  std::remove(s.begin(), s.end(), '\t');
  boost::to_upper(s);
  ....
}

Similar errors can be found in some other places:

V530 The return value of function 'remove' is required to be utilized. cmdline.cpp 107

ReactOS

V530 The return value of function 'wcscmp' is required to be utilized. dmoreg.c 621


#define strcmpW(s1,s2) wcscmp((s1),(s2))

static HRESULT WINAPI IEnumDMO_fnNext(....)
{
  ....
  if (Names[count])
    strcmpW(Names[count], szValue);
  ....
}

Chromium

V530 The return value of function 'remove_if' is required to be utilized. shortcuts_provider.cc 136


void ShortcutsProvider::DeleteMatchesWithURLs(
  const std::set<GURL>& urls)
{
  std::remove_if(matches_.begin(),
                 matches_.end(),
                 RemoveMatchPredicate(urls));
  listener_->OnProviderUpdate(true);
}

Multi Theft Auto

V530 The return value of function 'empty' is required to be utilized. cperfstat.functiontiming.cpp 197


std::map < SString, SFunctionTimingInfo > m_TimingMap;

void CPerfStatFunctionTimingImpl::DoPulse ( void )
{
  ....
  // Do nothing if not active
  if ( !m_bIsActive )
  {
    m_TimingMap.empty ();
    return;
  }
  ....
}

Multi Theft Auto

V530 The return value of function 'empty' is required to be utilized. cclientcolsphere.cpp 34


void CreateSphereFaces (
  std::vector < SFace >& faceList, int iIterations )
{
  int numFaces = (int)( pow ( 4.0, iIterations ) * 8 );
  faceList.empty ();
  faceList.reserve ( numFaces );
  ....
}

Similar errors can be found in some other places:

V530 The return value of function 'empty' is required to be utilized. cresource.cpp 132
V530 The return value of function 'empty' is required to be utilized. cresource.cpp 139
V530 The return value of function 'empty' is required to be utilized. cresource.cpp 147

Trans-Proteomic Pipeline

V530 The return value of function 'empty' is required to be utilized. tag.cxx 72


static void clearTagNames() {
   std::vector<const char *>ptrs;
   for (tagname_set::iterator i = tagnames.begin();
        i!=tagnames.end();i++) {
      ptrs.push_back(*i);
   }
   for (tagname_set::iterator j = attrnames.begin();
        j!=attrnames.end();j++) {
      ptrs.push_back(*j);
   }
   tagnames.empty();
   attrnames.empty();
   for (size_t n=ptrs.size();n--;) {
      delete [] (char *)(ptrs[n]); // cast away const
   }
}

Similar errors can be found in some other places:

V530 The return value of function 'empty' is required to be utilized. tag.cxx 73

QuantLib

V530 The return value of function 'empty' is required to be utilized. commoditycurve.cpp 70


void CommodityCurve::setPrices(std::map<Date, Real>& prices) {
  QL_REQUIRE(prices.size()>1, "too few prices");

  dates_.empty();
  data_.empty();

  for (std::map<Date, Real>::const_iterator i = prices.begin();
       i != prices.end(); i++)
  {
    dates_.push_back(i->first);
    data_.push_back(i->second);
  }
  ....
}

This is what should have been written here: dates_.clear(); data_.clear();

Similar errors can be found in some other places:

V530 The return value of function 'empty' is required to be utilized. commoditycurve.cpp 71

QuantLib

V530 The return value of function 'empty' is required to be utilized. energycommodity.cpp 152


void EnergyCommodity::calculateSecondaryCostAmounts(
  const CommodityType& commodityType,
  Real totalQuantityValue,
  const Date& evaluationDate) const
{
  secondaryCostAmounts_.empty();
  ....
}

QuantLib

V530 The return value of function 'remove' is required to be utilized. recursivecdoengine.hpp 172


template <class CDOEngine, class copulaT>
void RecursiveCdoEngine<CDOEngine, copulaT>::initialize() const {
  ....
  std::remove(lgds.begin(), lgds.end(), 0.);
  ....
}

Firebird

V530 The return value of function 'fgetpos' is required to be utilized. inputdevices.cpp 118


void InputDevices::indev::getPos(fpos_t* out) const
{
  fb_assert(out);
  fb_assert(indev_fpointer);
  fgetpos(indev_fpointer, out);
}

Synergy

V530 The return value of function 'empty' is required to be utilized. cipcserver.cpp 69


typedef std::list<CIpcClientProxy*> CClientList;
CClientList      m_clients;

CIpcServer::~CIpcServer()
{
  ....
  m_clients.empty();
  ....
}

Micro-Manager

V530 The return value of function 'c_str' is required to be utilized. ZeissCAN.cpp 1553


int HalogenLamp::SetIntensity(long intensity)
{
  ....
  command_stream.str().c_str();
  ....
}

Similar errors can be found in some other places:

V530 The return value of function 'c_str' is required to be utilized. ZeissCAN.cpp 2800

Unreal Engine 4

V530 The return value of function 'Memcmp' is required to be utilized. d3d11statecacheprivate.h 547


D3D11_STATE_CACHE_INLINE void GetBlendState(
  ID3D11BlendState** BlendState, float BlendFactor[4],
  uint32* SampleMask)
{
  ....
  FMemory::Memcmp(BlendFactor, CurrentBlendFactor,
                  sizeof(CurrentBlendFactor));
  ....
}

This is what should have been written here: FMemory::Memcpy

Qt

V530 The return value of function 'toLower' is required to be utilized. main.cpp 72


int main(int argc, char **argv)
{
  ....
  QByteArray arg(argv[a]);
  ....
  arg = arg.mid(1);
  arg.toLower();
  if (arg == "o")
  ....
}

Similar errors can be found in some other places:

V530 The return value of function 'toLower' is required to be utilized. main.cpp 1522

ITK

V530 The return value of function 'empty' is required to be utilized. itkpatchbaseddenoisingimagefilter.hxx 85


template <typename TInputImage, typename TOutputImage>
void
PatchBasedDenoisingImageFilter<TInputImage, TOutputImage>
::EmptyCaches()
{
  for (unsigned int threadId = 0;
       threadId < m_ThreadData.size(); ++threadId)
    {
    SizeValueType cacheSize = ....;
    for (SizeValueType c = 0; c < cacheSize; ++c)
      {
      delete m_ThreadData[threadId].eigenValsCache[c];
      delete m_ThreadData[threadId].eigenVecsCache[c];
      }
    m_ThreadData[threadId].eigenValsCache.empty();
    m_ThreadData[threadId].eigenVecsCache.empty();
    }
}

Similar errors can be found in some other places:

V530 The return value of function 'empty' is required to be utilized. itkpatchbaseddenoisingimagefilter.hxx 86

ITK

V530 The return value of function 'remove' is required to be utilized. itkge5imageio.cxx 297


GEImageHeader *
GE5ImageIO::ReadHeader(const char  *FileNameToRead)
{
  ....
  std::string tmpId(buffer+VOff(84,88), 13);
  std::remove(tmpId.begin(), tmpId.end(), '-');
  ....
}

ITK

V530 The return value of function 'empty' is required to be utilized. itkvideofilewriter.hxx 334


std::vector<SizeValueType>   m_Dimensions;

template< typename TInputVideoStream >
bool
VideoFileWriter< TInputVideoStream >
::InitializeOutputParameters()
{
  ....
  m_Dimensions.empty();
  ....
}

Spring Engine

V530 The return value of function 'size' is required to be utilized. assimp b3dimporter.cpp 536


void B3DImporter::ReadBB3D( aiScene *scene ){
  _textures.clear();
  _materials.size();     // <=
  _vertices.clear();
  _meshes.clear();
  ....
}

Doxygen

V530 The return value of function 'toupper' is required to be utilized. classdef.cpp 1963


int toupper (int __c);

void ClassDef::writeDocumentationContents(....)
{
  QCString pageType = " ";
  pageType += compoundTypeString();
  toupper(pageType.at(1));
  ....
}

FreeSWITCH

V530 The return value of function 'LoadLibraryExA' is required to be utilized. switch_dso.c 42


SWITCH_DECLARE(switch_dso_lib_t) switch_dso_open(....)
{
  HINSTANCE lib;

  lib = LoadLibraryEx(path, NULL, 0);

  if (!lib) {
    LoadLibraryEx(path, NULL, LOAD_WITH_ALTERED_SEARCH_PATH);
  }

  if (!lib) {
    DWORD error = GetLastError();
    *err = switch_mprintf("dll open error [%ul]\n", error);
  }

  return lib;
}

Mozilla Thunderbird

V530 The return value of function 'getenv' is required to be utilized. nswindowswmain.cpp 134


int wmain(int argc, WCHAR **argv)
{
  ....
  // Force creation of the multibyte _environ variable.
  getenv("PATH");
  int result = main(argc, argvConverted, _environ);
  ....
}

Wine Is Not an Emulator

V530 The return value of function 'DSCF_AddRef' is required to be utilized. dsound_main.c 760


static ULONG WINAPI DSCF_AddRef(LPCLASSFACTORY iface)
{
    return 2;
}

HRESULT WINAPI DllGetClassObject(....)
{
  ....
  while (NULL != DSOUND_CF[i].rclsid) {
    if (IsEqualGUID(rclsid, DSOUND_CF[i].rclsid)) {
      DSCF_AddRef(&DSOUND_CF[i].IClassFactory_iface);  // <=
      *ppv = &DSOUND_CF[i];
      return S_OK;
    }
    i++;
  }
  ....
}

Computational Network Toolkit

V530 The return value of function 'empty' is required to be utilized. utterancesourcemulti.h 340


template <class UTTREF>
std::vector<shiftedvector<....>>getclassids(const UTTREF &uttref)
{
  std::vector<shiftedvector<....>> allclassids;
  allclassids.empty();  // <=
  ....
}

Similar errors can be found in some other places:

V530 The return value of function 'empty' is required to be utilized. utterancesourcemulti.h 364

OpenToonz

V530 The return value of function 'toUpper' is required to be utilized. sceneviewerevents.cpp 847


void SceneViewer::keyPressEvent(QKeyEvent *event)
{
  ....
  QString text = event->text();
  if ((event->modifiers() & Qt::ShiftModifier))
    text.toUpper();
  ....
}

Similar errors can be found in some other places:

V530 The return value of function 'left' is required to be utilized. tfarmserver.cpp 569
V530 The return value of function 'ftell' is required to be utilized. tiio_bmp.cpp 804
V530 The return value of function 'accumulate' is required to be utilized. bendertool.cpp 374

ReactOS

V530 The return value of function 'GetWindow' is required to be utilized. oleobject.c 113


#define IOleInPlaceSite_GetWindow(This,phwnd)\
    ( (This)->lpVtbl -> GetWindow(This,phwnd) )

static void create_shell_embedding_hwnd(WebBrowser *This)
{
  HWND parent = NULL;
  ....
  if(SUCCEEDED(hres)) {
      IOleInPlaceSite_GetWindow(inplace, &parent); // <=
      IOleInPlaceSite_Release(inplace);
  }
  ....
}

Similar errors can be found in some other places:

V530 The return value of function 'GetWindow' is required to be utilized. oleobject.c 185

Open X-Ray Engine

V530 The return value of function 'empty' is required to be utilized. actor_network.cpp 657


BOOL CActor::net_Spawn(CSE_Abstract* DC)
{
  ....
  m_States.empty();
  ....
}

Similar errors can be found in some other places:

V530 The return value of function 'unique' is required to be utilized. uidragdroplistex.cpp 780

Casablanca

V530 The return value of function 'release' is required to be utilized. cpprestsdk140 http_server_api.cpp 64


std::unique_ptr<http_server>
  http_server_api::s_server_api((http_server*)nullptr);

void http_server_api::unregister_server_api()
{
  pplx::extensibility::scoped_critical_section_t lock(s_lock);

  if (http_server_api::has_listener())
  {
    throw http_exception(_XPLATSTR("Server API ..... attached"));
  }

  s_server_api.release();
}

CodeLite

V530 The return value of function 'empty' is required to be utilized. tokenizer.cpp 56


StringTokenizer::StringTokenizer(const wxString& str,
                const wxString& strDelimiter,
                const bool &bAllowEmptyTokens /* false */)
{
  ....
  wxString token;
  while( nEnd != -1 )
  {
    if( nEnd != nStart)
      token = str.substr(nStart, nEnd-nStart);
    else
      token.empty();                                       // <=

    if(!token.empty())
      m_tokensArr.push_back(token);
    ....
  }
}

LLVM/Clang

V530 The return value of function 'release' is required to be utilized. VerifyDiagnosticConsumer.cpp 46


std::unique_ptr<DiagnosticConsumer> takeClient()
  { return std::move(Owner); }

VerifyDiagnosticConsumer::~VerifyDiagnosticConsumer() {
  ....
  SrcManager = nullptr;
  CheckDiagnostics();
  Diags.takeClient().release();
}

CryEngine V

V530 The return value of function 'release' is required to be utilized. ClipVolumes.cpp 492


vector<unique_ptr<CFullscreenPass>> m_jitteredDepthPassArray;

void CClipVolumesStage::PrepareVolumetricFog()
{
  ....
  for (int32 i = 0; i < m_jitteredDepthPassArray.size(); ++i)
  {
    m_jitteredDepthPassArray[i].release();
  }

  m_jitteredDepthPassArray.resize(depth);

  for (int32 i = 0; i < depth; ++i)
  {
    m_jitteredDepthPassArray[i] = CryMakeUnique<....>();
    m_jitteredDepthPassArray[i]->SetViewport(viewport);
    m_jitteredDepthPassArray[i]->SetFlags(....);
  }
  ....
}

Scilab

V530 The return value of function 'back' is required to be utilized. sci_mscanf.cpp 274


types::Function::ReturnValue sci_mscanf(....)
{
  ....
  std::vector<types::InternalType*> pITTemp = std::vector<...>();
  ....
  case types::InternalType::ScilabString :
  {
    ....
    pITTemp.pop_back();       // <=
    pITTemp.push_back(pType);
  }
  break;
  case types::InternalType::ScilabDouble :
  {
    ....
    pITTemp.back();           // <= ???
    pITTemp.push_back(pType);
  }
  break;
  ....
}

Audacity

V530 The return value of function 'remove' is required to be utilized. OverlayPanel.cpp 31


bool OverlayPanel::RemoveOverlay(Overlay *pOverlay)
{
  const size_t oldSize = mOverlays.size();
  std::remove(mOverlays.begin(), mOverlays.end(), pOverlay);
  return oldSize != mOverlays.size();
}

Audacity

V530 The return value of function 'Left' is required to be utilized. ASlider.cpp 973


wxString LWSlider::GetTip(float value) const
{
  wxString label;

  if (mTipTemplate.IsEmpty())
  {
    wxString val;

    switch(mStyle)
    {
    case FRAC_SLIDER:
      val.Printf(wxT("%.2f"), value);
      break;

    case DB_SLIDER:
      val.Printf(wxT("%+.1f dB"), value);
      if (val.Right(1) == wxT("0"))
      {
        val.Left(val.Length() - 2);        // <=
      }
      break;
  ....
}

wxString Left (size_t count) const

Ardour

V530 The return value of function 'unique' is required to be utilized. audio_library.cc 162


void
AudioLibrary::search_members_and (vector<string>& members,
const vector<string>& tags)
{
  ....
  sort(members.begin(), members.end());
  unique(members.begin(), members.end());
  ....
}

sort(members.begin(), members.end()); auto last = unique(members.begin(), members.end()); v.erase(last, members.end());

Chromium

V530 CWE-252 The return value of function 'remove_if' is required to be utilized. pdf_to_emf_converter.cc 44


void OnConvertedClientDisconnected() {
  // We have no direct way of tracking which
  // PdfToEmfConverterClientPtr got disconnected as it is a
  // movable type, short of using a wrapper.
  // Just traverse the list of clients and remove the ones
  // that are not bound.

  std::remove_if(
    g_converter_clients.Get().begin(),
    g_converter_clients.Get().end(),
    [](const mojom::PdfToEmfConverterClientPtr& client) {
      return !client.is_bound();
    });
}

A function remove_if deletes nothing, but only rearranges the elements in a container.

Android

V530 CWE-252 The return value of function 'bswap32' is required to be utilized. ELFAttribute.cpp 84


inline uint32_t bswap32(uint32_t pData) {
  return
    (((pData & 0xFF000000) >> 24) | ((pData & 0x00FF0000) >> 8) |
     ((pData & 0x0000FF00) << 8) | ((pData & 0x000000FF) << 24));
}

bool ELFAttribute::merge(....) {
  ....
  uint32_t subsection_length =
    *reinterpret_cast<const uint32_t*>(subsection_data);

  if (llvm::sys::IsLittleEndianHost !=
      m_Config.targets().isLittleEndian())
    bswap32(subsection_length);               // <=
  ....
}

Similar errors can be found in some other places:

V530 CWE-252 The return value of function 'bswap32' is required to be utilized. ELFAttribute.cpp 218
V530 CWE-252 The return value of function 'bswap32' is required to be utilized. ELFAttribute.cpp 346
V530 CWE-252 The return value of function 'bswap32' is required to be utilized. ELFAttribute.cpp 352

ANGLE

V530 CWE-252 The return value of function 'release' is required to be utilized. error.inl 52


class Error final
{
  ....
  mutable std::unique_ptr<std::string> mMessage;
};

Error &Error::operator=(const Error &other)
{
  mCode = other.mCode;
  mID = other.mID;

  if (other.mMessage)
  {
    createMessageString();
    *mMessage = *(other.mMessage);
  }
  else
  {
    mMessage.release();
  }

  return *this;
}

Similar errors can be found in some other places:

V530 CWE-252 The return value of function 'release' is required to be utilized. error.inl 126

Qt

V530 CWE-252 The return value of function 'InitializeAcl' is required to be utilized. qlocalserver_win.cpp 144


bool QLocalServerPrivate::addListener()
{
  ....
  InitializeAcl(acl, aclSize, ACL_REVISION_DS);
  ....
}

Qt

V530 CWE-252 The return value of function 'SetSecurityDescriptorOwner' is required to be utilized. qlocalserver_win.cpp 167


bool QLocalServerPrivate::addListener()
{
  ....
  SetSecurityDescriptorOwner(pSD.data(), pTokenUser->User.Sid, FALSE);
  SetSecurityDescriptorGroup(pSD.data(), pTokenGroup->PrimaryGroup, FALSE);
  ....
}

Similar errors can be found in some other places:

V530 CWE-252 The return value of function 'SetSecurityDescriptorGroup' is required to be utilized. qlocalserver_win.cpp 168

LibreOffice

V530 The return value of function 'SysAllocString' is required to be utilized. maccessible.cxx 1077


STDMETHODIMP CMAccessible::put_accValue(....)
{
  ....
  if(varChild.lVal==CHILDID_SELF)
  {
    SysAllocString(m_pszValue);
    m_pszValue=SysAllocString(szValue);
    return S_OK;
  }
  ....
}

Haiku Operation System

V530 The return value of function 'begin' is required to be utilized. IMAPFolder.cpp 414


void
IMAPFolder::RegisterPendingBodies(...., const BMessenger* replyTo)
{
  ....
  IMAP::MessageUIDList::const_iterator iterator = uids.begin();
  for (; iterator != uids.end(); iterator++) {
    if (replyTo != NULL)
      fPendingBodies[*iterator].push_back(*replyTo);
    else
      fPendingBodies[*iterator].begin();   // <=
  }
}

Blender

V530 The return value of function 'clamp' is required to be utilized. BLI_math_vector.hh 88


template <typename T, int Size>
inline vec_base<T, Size>
  clamp(const vec_base<T, Size> &a, const T &min, const T &max)
{
  vec_base<T, Size> result = a;
  for (int i = 0; i < Size; i++) {
    std::clamp(result[i], min, max);
  }
  return result;
}

Ogre3D

V530 The return value of function 'back' is required to be utilized. OgreGLXConfigDialog.cpp 410


class GLXConfigurator
{
public:
  // ....
  std::list<ConfigCallbackData> mConfigCallbackData
  // ....
}
void GLXConfigurator::SetRenderer(RenderSystem *r)

  // ....
  mConfigCallbackData.back();
  // ....
}

Here, for some reason, we call the back function of the std::list container to get a reference to the last element. However, we do not use or save this reference.

LLVM/Clang

V530 The return value of function 'getRangeRef' is required to be utilized. ScalarEvolution.cpp:6587


ScalarEvolution::getRangeRefIter(const SCEV *S,
                                 ScalarEvolution::RangeSignHint SignHint)
{
  ....
  for (const SCEV *P : reverse(drop_begin(WorkList))) {
    getRangeRef(P, SignHint);
    ....
  }
  ....
}

Your contact information:

Desired license type:

Examples of errors detected by the V530 diagnostic

TortoiseSVN

TortoiseSVN

TortoiseSVN

WinMerge

WinMerge

EchoVNC

EchoVNC

IPP Samples

Nmap Security Scanner

Chromium

Chromium

Intel AMT SDK

Intel AMT SDK

ReactOS

Battle for Wesnoth

wxWidgets

wxWidgets

Visualization Toolkit (VTK)

MongoDB

ReactOS

Chromium

Multi Theft Auto

Multi Theft Auto

Trans-Proteomic Pipeline

QuantLib

QuantLib

QuantLib

Firebird

Synergy

Micro-Manager

Unreal Engine 4

Qt

ITK

ITK

ITK

Spring Engine

Doxygen

FreeSWITCH

Mozilla Thunderbird

Wine Is Not an Emulator

Computational Network Toolkit

OpenToonz

ReactOS

Open X-Ray Engine

Casablanca

CodeLite

LLVM/Clang

CryEngine V

Scilab

Audacity

Audacity

Ardour

Chromium

Android

ANGLE

Qt

Qt

LibreOffice

Haiku Operation System

Blender

Ogre3D

LLVM/Clang

Want to try PVS‑Studio for free?