Ticket #5023: VS2008StaticCodeAnalyzer.patch

File VS2008StaticCodeAnalyzer.patch, 17.0 KB (added by eyebex, 12 years ago)
  • AsyncSocketEx.cpp

     
    805805#ifndef NOLAYERS
    806806    if (m_pFirstLayer)
    807807    {
    808         BOOL res = m_pFirstLayer->Create(nSocketPort, nSocketType, lEvent, lpszSocketAddress, nFamily, reusable);
     808        res = m_pFirstLayer->Create(nSocketPort, nSocketType, lEvent, lpszSocketAddress, nFamily, reusable);
    809809#ifndef NOSOCKETSTATES
    810810        if (res)
    811811            SetState(unconnected);
     
    13001300
    13011301        addrinfo hints;
    13021302        int error;
    1303         BOOL ret;
     1303        BOOL ret = FALSE;
    13041304        char port[10];
    13051305
    13061306        memset(&hints, 0, sizeof(addrinfo));
     
    14251425        return m_pFirstLayer->GetPeerName(rPeerAddress, rPeerPort);
    14261426#endif NOLAYERS
    14271427
    1428     SOCKADDR* sockAddr;
    1429     int nSockAddrLen;
     1428    SOCKADDR* sockAddr = NULL;
     1429    int nSockAddrLen = 0;
    14301430
    14311431    if (m_SocketData.nFamily == AF_INET6)
    14321432    {
     
    14851485#ifdef _AFX
    14861486BOOL CAsyncSocketEx::GetSockName(CString& rSocketAddress, UINT& rSocketPort)
    14871487{
    1488     SOCKADDR* sockAddr;
    1489     int nSockAddrLen;
     1488    SOCKADDR* sockAddr = NULL;
     1489    int nSockAddrLen = 0;
    14901490
    14911491    if (m_SocketData.nFamily == AF_INET6)
    14921492    {
  • AsyncSocketExLayer.cpp

     
    483483        return m_pNextLayer->GetPeerName(rPeerAddress, rPeerPort);
    484484    else
    485485    {
    486         SOCKADDR* sockAddr;
    487         int nSockAddrLen;
     486        SOCKADDR* sockAddr = NULL;
     487        int nSockAddrLen = 0;
    488488
    489489        if (m_nFamily == AF_INET6)
    490490        {
     
    562562        return m_pNextLayer->GetSockName(rSockAddress, rSockPort);
    563563    else
    564564    {
    565         SOCKADDR* sockAddr;
    566         int nSockAddrLen;
     565        SOCKADDR* sockAddr = NULL;
     566        int nSockAddrLen = 0;
    567567
    568568        if (m_nFamily == AF_INET6)
    569569        {
  • AsyncSslSocketLayer.cpp

     
    505505            // will stall. Perhaps it triggers some internal processing.
    506506            // Also, ignore return value, don't do any error checking. This function
    507507            // can report errors, even though a later call can succeed.
    508             char buffer;
    509             pBIO_read(m_sslbio, &buffer, 0);
     508            char dummy;
     509            pBIO_read(m_sslbio, &dummy, 0);
    510510        }
    511511        else if (!numread)
    512512        {
  • Interface/AdminSocket.cpp

     
    181181            }
    182182            if (m_nRecvBufferPos < 5)
    183183                return FALSE;
    184             int len = m_pRecvBuffer[3] * 256 + m_pRecvBuffer[4];
     184            len = m_pRecvBuffer[3] * 256 + m_pRecvBuffer[4];
    185185            if (len != 4)
    186186            {
    187187                CString str;
  • Interface/ConnectDialog.cpp

     
    7373{
    7474    CDialog::OnInitDialog();
    7575   
    76     m_ServerAddress = m_pOptions->GetOption(IOPTION_LASTSERVERADDRESS);
    77     m_ServerPort.Format(_T("%d"), m_pOptions->GetOptionVal(IOPTION_LASTSERVERPORT));
    78     m_ServerPassword    = m_pOptions->GetOption(IOPTION_LASTSERVERPASS);
     76    m_ServerAddress = m_pOptions->GetOption(IOPTION_LASTSERVERADDRESS);
     77    m_ServerPort.Format(_T("%d"), static_cast<int>(m_pOptions->GetOptionVal(IOPTION_LASTSERVERPORT)));
     78    m_ServerPassword = m_pOptions->GetOption(IOPTION_LASTSERVERPASS);
    7979    m_bAlways = m_pOptions->GetOptionVal(IOPTION_ALWAYS) != 0;
    8080    UpdateData(FALSE);
    8181
  • Interface/GroupsDlg.cpp

     
    330330
    331331            if (!dlg.m_action)
    332332            {
    333                 for (unsigned int i = 0; i < m_UsersList.size(); i++)
     333                for (unsigned int k = 0; k < m_UsersList.size(); k++)
    334334                {
    335                     t_user& user = m_UsersList[i];
     335                    t_user& user = m_UsersList[k];
    336336                    if (user.group.CompareNoCase(oldName))
    337337                        continue;
    338338                    user.group = dlg.m_groupName;
     
    342342            {
    343343                // Delete affected users
    344344                t_UsersList newList;
    345                 for (unsigned int i = 0; i < m_UsersList.size(); i++)
     345                for (unsigned int k = 0; k < m_UsersList.size(); k++)
    346346                {
    347                     if (m_UsersList[i].group.CompareNoCase(oldName))
    348                         newList.push_back(m_UsersList[i]);
     347                    if (m_UsersList[k].group.CompareNoCase(oldName))
     348                        newList.push_back(m_UsersList[k]);
    349349                }
    350350                m_UsersList = newList;
    351351            }
  • Interface/misc/BrowseForFolder.cpp

     
    114114    if ((pidl = ::SHBrowseForFolder(&m_bi)) != NULL)
    115115    {
    116116        m_strPath.Empty();
    117         if (SUCCEEDED(::SHGetPathFromIDList(pidl, m_szSelected)))
     117        if (::SHGetPathFromIDList(pidl, m_szSelected))
    118118        {
    119119            bRet = true;
    120120            m_strPath = m_szSelected;
  • Interface/misc/ExceptionReport.cpp

     
    254254        }
    255255
    256256        int nError=0;
    257         if (m_hReportFile == INVALID_HANDLE_VALUE && INVALID_HANDLE_VALUE)
     257        if (m_hReportFile == INVALID_HANDLE_VALUE)
    258258            nError = GetLastError();
    259259        else
    260260        {
  • Interface/misc/HyperLink.cpp

     
    458458            HCURSOR hHandCursor = ::LoadCursor(hModule, MAKEINTRESOURCE(106));
    459459            if (hHandCursor)
    460460                g_hLinkCursor = CopyCursor(hHandCursor);
     461            FreeLibrary(hModule);
    461462        }
    462         FreeLibrary(hModule);
    463463    }
    464464}
    465465
  • Interface/misc/MailMsg.cpp

     
    203203        message.flFlags                           = 0;
    204204        message.lpOriginator                      = m_from.size() ? pRecipients : NULL;
    205205        message.nRecipCount                       = nRecipients - m_from.size(); // don't count originator
    206         message.lpRecips                          = nRecipients - m_from.size() ? &pRecipients[m_from.size()] : NULL;
     206        message.lpRecips                          = (nRecipients - m_from.size()) ? &pRecipients[m_from.size()] : NULL;
    207207        message.nFileCount                        = nAttachments;
    208208        message.lpFiles                           = nAttachments ? pAttachments : NULL;
    209209
  • Interface/misc/SAPrefsDialog.cpp

     
    229229    else
    230230    {
    231231        // find start page
    232         for (unsigned int i = 0; i <m_pages.size(); i++)
     232        for (i = 0; i <m_pages.size(); i++)
    233233        {
    234234            pageStruct *pPS = m_pages[i];
    235235            ASSERT(pPS);
  • Interface/OptionsAdminInterfacePage.cpp

     
    249249
    250250void COptionsAdminInterfacePage::LoadData()
    251251{
    252     m_Port.Format(_T("%d"), m_pOptionsDlg->GetOptionVal(OPTION_ADMINPORT));
     252    m_Port.Format(_T("%d"), static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_ADMINPORT)));
    253253    m_IpBindings = m_pOptionsDlg->GetOption(OPTION_ADMINIPBINDINGS);
    254254    m_IpAddresses = m_pOptionsDlg->GetOption(OPTION_ADMINIPADDRESSES);
    255255    m_NewPass = m_pOptionsDlg->GetOption(OPTION_ADMINPASS);
  • Interface/OptionsAutobanPage.cpp

     
    7878void COptionsAutobanPage::LoadData()
    7979{
    8080    m_enable = m_pOptionsDlg->GetOptionVal(OPTION_AUTOBAN_ENABLE) ? true : false;
    81     m_attempts.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_AUTOBAN_ATTEMPTS));
    82     m_time.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_AUTOBAN_BANTIME));
     81    m_attempts.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_AUTOBAN_ATTEMPTS)));
     82    m_time.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_AUTOBAN_BANTIME)));
    8383    m_type = m_pOptionsDlg->GetOptionVal(OPTION_AUTOBAN_TYPE) ? 1 : 0;
    8484}
    8585
  • Interface/OptionsCompressionPage.cpp

     
    111111
    112112void COptionsCompressionPage::LoadData()
    113113{
    114     m_LevelMin.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_MODEZ_LEVELMIN));
    115     m_LevelMax.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_MODEZ_LEVELMAX));
     114    m_LevelMin.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_MODEZ_LEVELMIN)));
     115    m_LevelMax.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_MODEZ_LEVELMAX)));
    116116    m_UseModeZ = m_pOptionsDlg->GetOptionVal(OPTION_MODEZ_USE) != 0;
    117117    m_DisallowLocal = !m_pOptionsDlg->GetOptionVal(OPTION_MODEZ_ALLOWLOCAL);
    118118    m_disallowedIPs =  m_pOptionsDlg->GetOption(OPTION_MODEZ_DISALLOWED_IPS);
  • Interface/OptionsDlg.cpp

     
    369369                char* utf8 = ConvToNetwork(GetOption(i+1));
    370370                if (utf8)
    371371                {
    372                     int len = strlen(utf8);
    373                     *p++ = (len / 256) / 256;
    374                     *p++ = len / 256;
    375                     *p++ = len % 256;
    376                     memcpy(p, utf8, len);
    377                     p += len;
     372                    int l = strlen(utf8);
     373                    *p++ = (l / 256) / 256;
     374                    *p++ = l / 256;
     375                    *p++ = l % 256;
     376                    memcpy(p, utf8, l);
     377                    p += l;
    378378                    delete [] utf8;
    379379                }
    380380                else
  • Interface/OptionsGeneralPage.cpp

     
    144144
    145145void COptionsGeneralPage::LoadData()
    146146{
    147     m_Port = m_pOptionsDlg->GetOption(OPTION_SERVERPORT);   
    148     m_Threadnum.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_THREADNUM));   
    149     m_MaxUsers.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_MAXUSERS)); 
    150     m_Timeout.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_TIMEOUT));   
    151     m_NoTransferTimeout.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_NOTRANSFERTIMEOUT));
    152     m_LoginTimeout.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_LOGINTIMEOUT));
     147    m_Port = m_pOptionsDlg->GetOption(OPTION_SERVERPORT);
     148    m_Threadnum.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_THREADNUM)));
     149    m_MaxUsers.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_MAXUSERS)));
     150    m_Timeout.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_TIMEOUT)));
     151    m_NoTransferTimeout.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_NOTRANSFERTIMEOUT)));
     152    m_LoginTimeout.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_LOGINTIMEOUT)));
    153153}
    154154
    155155void COptionsGeneralPage::SaveData()
  • Interface/OptionsMiscPage.cpp

     
    8484{
    8585    m_bDontShowPass = m_pOptionsDlg->GetOptionVal(OPTION_LOGSHOWPASS) == 0;
    8686    m_bStartMinimized = m_pOptionsDlg->m_pInterfaceOptions->GetOptionVal(IOPTION_STARTMINIMIZED) != 0;
    87     m_TransferBufferSize.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_BUFFERSIZE));
    88     m_TransferBufferSize2.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_BUFFERSIZE2));
     87    m_TransferBufferSize.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_BUFFERSIZE)));
     88    m_TransferBufferSize2.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_BUFFERSIZE2)));
    8989    m_bSharedWrite = m_pOptionsDlg->GetOptionVal(OPTION_SHAREDWRITE) != 0;
    9090    m_bActiveIgnoreLocal = m_pOptionsDlg->GetOptionVal(OPTION_ACTIVE_IGNORELOCAL) != 0;
    9191}
  • Interface/OptionsPasvPage.cpp

     
    162162    m_IP = m_pOptionsDlg->GetOption(OPTION_CUSTOMPASVIP);
    163163    m_URL = m_pOptionsDlg->GetOption(OPTION_CUSTOMPASVIPSERVER);
    164164    m_bUseCustomPort = m_pOptionsDlg->GetOptionVal(OPTION_USECUSTOMPASVPORT) != 0;
    165     m_PortMin.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_CUSTOMPASVMINPORT));
    166     m_PortMax.Format("%d", m_pOptionsDlg->GetOptionVal(OPTION_CUSTOMPASVMAXPORT));
     165    m_PortMin.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_CUSTOMPASVMINPORT)));
     166    m_PortMax.Format("%d", static_cast<int>(m_pOptionsDlg->GetOptionVal(OPTION_CUSTOMPASVMAXPORT)));
    167167    m_NoExternalOnLocal = m_pOptionsDlg->GetOptionVal(OPTION_NOEXTERNALIPONLOCAL) != 0;
    168168}
    169169
  • Interface/UsersListCtrl.cpp

     
    300300        CConnectionData *pConnectionData = iter->second;
    301301
    302302        m_connectionDataMap.erase(iter);
    303         for (std::vector<CConnectionData*>::iterator iter = m_connectionDataArray.begin() + pConnectionData->listIndex + 1; iter != m_connectionDataArray.end(); iter++)
    304             (*iter)->listIndex--;
     303        for (std::vector<CConnectionData*>::iterator i = m_connectionDataArray.begin() + pConnectionData->listIndex + 1; i != m_connectionDataArray.end(); i++)
     304            (*i)->listIndex--;
    305305        m_connectionDataArray.erase(m_connectionDataArray.begin() + pConnectionData->listIndex);
    306306        delete pConnectionData;
    307307
  • version.cpp

     
    8383                {
    8484                    TCHAR ch = 'a';
    8585                    ch += static_cast<TCHAR>(fi->dwFileVersionLS >> 16) - 1;
    86                     version.Format(_T("%s version %d.%d%c test release %d)"), ProductName, fi->dwFileVersionMS >> 16, fi->dwFileVersionMS & 0xFFFF, ch, fi->dwFileVersionLS & 0xFFFF);
     86                    version.Format(_T("%s version %d.%d%c test release %d)"), ProductName.GetString(), fi->dwFileVersionMS >> 16, fi->dwFileVersionMS & 0xFFFF, ch, fi->dwFileVersionLS & 0xFFFF);
    8787                }
    8888                else
    89                     version.Format(_T("%s version %d.%d test release %d"), ProductName, fi->dwFileVersionMS >> 16, fi->dwFileVersionMS & 0xFFFF, fi->dwFileVersionLS & 0xFFFF);
     89                    version.Format(_T("%s version %d.%d test release %d"), ProductName.GetString(), fi->dwFileVersionMS >> 16, fi->dwFileVersionMS & 0xFFFF, fi->dwFileVersionLS & 0xFFFF);
    9090            }
    9191            else
    9292            { //final versions
     
    9494                {
    9595                    TCHAR ch = 'a';
    9696                    ch += static_cast<TCHAR>(fi->dwFileVersionLS >> 16) - 1;
    97                     version.Format(_T("%s version %d.%d%c final"),ProductName, fi->dwFileVersionMS >> 16, fi->dwFileVersionMS & 0xFFFF, ch);
     97                    version.Format(_T("%s version %d.%d%c final"), ProductName.GetString(), fi->dwFileVersionMS >> 16, fi->dwFileVersionMS & 0xFFFF, ch);
    9898                }
    9999                else
    100                     version.Format(_T("%s version %d.%d final"), ProductName, fi->dwFileVersionMS >> 16, fi->dwFileVersionMS & 0xFFFF);
     100                    version.Format(_T("%s version %d.%d final"), ProductName.GetString(), fi->dwFileVersionMS >> 16, fi->dwFileVersionMS & 0xFFFF);
    101101            }
    102102        }
    103103        else
     
    106106            if ((fi->dwFileVersionLS & 0xFFFF) / 100)
    107107                if ((fi->dwFileVersionLS & 0xFFFF) % 100)
    108108                    //test release
    109                     version.Format(_T("%s version 0.%d.%d%c beta test release %d"), ProductName, fi->dwFileVersionMS & 0xFFFF, fi->dwFileVersionLS >> 16, (fi->dwFileVersionLS & 0xFFFF) / 100 + 'a' - 1, (fi->dwFileVersionLS & 0xFFFF) % 100);
     109                    version.Format(_T("%s version 0.%d.%d%c beta test release %d"), ProductName.GetString(), fi->dwFileVersionMS & 0xFFFF, fi->dwFileVersionLS >> 16, (fi->dwFileVersionLS & 0xFFFF) / 100 + 'a' - 1, (fi->dwFileVersionLS & 0xFFFF) % 100);
    110110                else
    111111                    //final version
    112                     version.Format(_T("%s version 0.%d.%d%c beta"),ProductName, fi->dwFileVersionMS & 0xFFFF, fi->dwFileVersionLS>>16, (fi->dwFileVersionLS & 0xFFFF) / 100 + 'a' - 1);
     112                    version.Format(_T("%s version 0.%d.%d%c beta"), ProductName.GetString(), fi->dwFileVersionMS & 0xFFFF, fi->dwFileVersionLS>>16, (fi->dwFileVersionLS & 0xFFFF) / 100 + 'a' - 1);
    113113            else
    114114                if (fi->dwFileVersionLS&0xFFFF)
    115115                    //test release
    116                     version.Format(_T("%s version 0.%d.%d beta test release %d"), ProductName,fi->dwFileVersionMS & 0xFFFF, fi->dwFileVersionLS >> 16, fi->dwFileVersionLS & 0xFFFF);
     116                    version.Format(_T("%s version 0.%d.%d beta test release %d"), ProductName.GetString(), fi->dwFileVersionMS & 0xFFFF, fi->dwFileVersionLS >> 16, fi->dwFileVersionLS & 0xFFFF);
    117117                else
    118118                    //final version
    119                     version.Format(_T("%s version 0.%d.%d beta"), ProductName, fi->dwFileVersionMS&0xFFFF, fi->dwFileVersionLS >> 16);
     119                    version.Format(_T("%s version 0.%d.%d beta"), ProductName.GetString(), fi->dwFileVersionMS&0xFFFF, fi->dwFileVersionLS >> 16);
    120120        }
    121121       
    122122    }