Embarcadero C ++ बिल्डर द्वारा विकसित एक ओपन WinSCP प्रोजेक्ट को मान्य करना

हम लगातार C / C ++ में ओपन सोर्स प्रोजेक्ट्स की जांच करते हैं। लेकिन लगभग हमेशा, ये विजुअल स्टूडियो में विकसित परियोजनाएं हैं। लेकिन हम किसी तरह से Embarcadero C ++ बिल्डर से वंचित हो गए। हमें इसे ठीक करने की आवश्यकता है और आज हमने WinSCP परियोजना की जाँच की।

WinSCP

WinSCP विंडोज के लिए डिजाइन किए गए SFTP और SCP प्रोटोकॉल के लिए एक फ्री ग्राफिकल क्लाइंट है। जीएनयू जीपीएल के तहत वितरित किया गया। इन प्रोटोकॉल का समर्थन करने वाले कंप्यूटर और सर्वर के बीच सुरक्षित फाइल कॉपी प्रदान करता है।



आधिकारिक वेबसाइट: http://winscp.net



प्रोजेक्ट बनाने के लिए, आपको Embarcadero C ++ Builder XE2 की आवश्यकता है।

निरीक्षण

पीवीएस-स्टूडियो विश्लेषक का उपयोग करके सत्यापन किया गया था। फिलहाल, पीवीएस-स्टूडियो समर्थन करता है:

• विजुअल स्टूडियो 2013 C, C ++, C ++ 11, C ++ / CX (WinRT)
• विजुअल स्टूडियो 2012 C, C ++, C ++ 11, C ++ / CX (WinRT)
• विजुअल स्टूडियो 2010 सी, सी ++, सी ++ 0x
• विजुअल स्टूडियो 2008 सी, सी ++
• विजुअल स्टूडियो 2005 सी, सी ++
• Embarcadero RAD Studio XE5 C, C ++, C ++ 11, 64-बिट कंपाइलर शामिल हैं
• Embarcadero RAD Studio XE4 C, C ++, C ++ 11, 64-बिट कंपाइलर शामिल हैं
• Embarcadero RAD Studio XE3 अपडेट 1 सी, सी ++, सी ++ 11, 64-बिट संकलक शामिल हैं
• एम्बरकैरो राड स्टूडियो एक्सई 2 सी, सी ++, सी ++ 0x
• एम्बरकैडरो राड स्टूडियो एक्सई सी, सी ++
• एम्बरकैडरो राड स्टूडियो 2010 सी, सी ++
• एम्बरकैडरो राड स्टूडियो 2009 सी, सी ++
• न्यूनतम सी, सी ++, सी ++ 11

साथ ही आप पीवीएस-स्टूडियो स्टैंडअलोन चला सकते हैं। यह आपको पूर्व-तैयार * .i फ़ाइलों की जांच करने या किसी परियोजना की निर्माण प्रक्रिया को ट्रैक करने और सत्यापन के लिए आवश्यक सभी जानकारी एकत्र करने की अनुमति देता है। विवरण यहां पाया जा सकता है: " पीवीएस-स्टूडियो अब किसी भी निर्माण प्रणाली का समर्थन करता है ।"

मान्यता परिणाम

कई त्रुटियां नहीं थीं, लेकिन इस लेख को लिखने और Embarcadero RAD Studio उपयोगकर्ताओं का ध्यान आकर्षित करने के लिए पर्याप्त है।

मेमेंट () फ़ंक्शन के लिए तर्कों को भ्रमित किया

TForm * __fastcall TMessageForm::Create(....) { .... LOGFONT AFont; .... memset(&AFont, sizeof(AFont), 0); .... }
      
      

        
        
        
      

    
        
        
        
      
      

        
        
        
      

    
     

PVS- स्टूडियो चेतावनी: V575 'मेमसेट' फ़ंक्शन '0' तत्वों को प्रोसेस करता है। तीसरे तर्क का निरीक्षण करें। messagedlg.cpp 786



तीसरे तर्क में मेमसेट () फ़ंक्शन सरणी का आकार लेता है। सरल लेकिन अप्रिय टाइपो। संरचना असमान है।



नीचे दिए गए कोड में एक और समान टाइपो है: messagedlg.cpp 796

एक नगण्य वस्तु का उपयोग करना

 void __fastcall TCustomScpExplorerForm::EditorAutoConfig() { .... else { .... TEditorList EditorList; EditorList = *WinConfiguration->EditorList; EditorList.Insert(0, new TEditorPreferences(EditorData)); WinConfiguration->EditorList = &EditorList; } .... }
      
      

        
        
        
      

    
        
        
        
      
      

        
        
        
      

    
     

PVS-Studio चेतावनी: V506 पॉइंटर को स्थानीय चर 'EditorList' में इस चर के दायरे से बाहर रखा गया है। ऐसा पॉइंटर अमान्य हो जाएगा। customscpexplorer.cpp 2633



स्कोप छोड़ने के तुरंत बाद 'EditorList' ऑब्जेक्ट नष्ट हो जाएगा। हालाँकि, प्रोग्राम इस ऑब्जेक्ट के लिए एक पॉइंटर को स्टोर करता है और फिर उसका उपयोग करता है। इससे अपरिभाषित व्यवहार होता है।

संवाद में कचरा

 bool __fastcall RecursiveDeleteFile(....) { SHFILEOPSTRUCT Data; memset(&Data, 0, sizeof(Data)); .... Data.pTo = L""; .... }
      
      

        
        
        
      

    
        
        
        
      
      

        
        
        
      

    
     

PVS-Studio चेतावनी: V540 सदस्य 'pTo' को दो 0 वर्णों के द्वारा स्ट्रिंग को इंगित करना चाहिए। 16.c पर आम



MSDN पर pTo पैरामीटर के विवरण में निम्नलिखित पंक्ति को नोट करें: "नोट यह स्ट्रिंग डबल-शून्य समाप्त होना चाहिए।"



फ़ाइल के साथ काम करने के लिए संवाद में त्रुटि के कारण कचरा समाहित हो जाएगा। या नहीं होगा। यह सब भाग्य पर निर्भर करता है। लेकिन कोड वैसे भी गलत है।

डुप्लीकेट पंक्ति

 int CFileZillaApi::Init(....) { .... m_pMainThread->m_hOwnerWnd=m_hOwnerWnd; m_pMainThread->m_hOwnerWnd=m_hOwnerWnd; .... }
      
      

        
        
        
      

    
        
        
        
      
      

        
        
        
      

    
     

PVS-Studio चेतावनी: V519 'm_pMainThread-> m_hOwnerWnd' चर को क्रमिक रूप से दो बार मान दिया गया है। शायद यह एक गलती है। जाँच लाइनें: 88, 89. filezillaapi.cpp 89



शायद कोई गलती नहीं है। बस एक लाइन सतही है।

टूटी हुई जाँच

 STDMETHODIMP CShellExtClassFactory::CreateInstance(....) { .... CShellExt* ShellExt = new CShellExt(); if (NULL == ShellExt) { return E_OUTOFMEMORY; } .... }
      
      

        
        
        
      

    
        
        
        
      
      

        
        
        
      

    
     

PVS-Studio चेतावनी: V668 'शेलएक्स्ट' पॉइंटर को अशक्त करने के परीक्षण में कोई समझदारी नहीं है, क्योंकि स्मृति को 'नए' ऑपरेटर का उपयोग करके आवंटित किया गया था। मेमोरी आवंटन त्रुटि के मामले में अपवाद उत्पन्न होगा। dragext.cpp 554



"If (NULL == शेलएक्स्ट)" को चेक करने का मतलब नहीं है, क्योंकि यदि मेमोरी आवंटित नहीं की जा सकती है, तो 'नया' ऑपरेटर अपवाद std :: bad_alloc फेंक देगा।

Fprintf () फ़ंक्शन का उपयोग करने का खतरनाक तरीका

 bool CAsyncSslSocketLayer::CreateSslCertificate(....) { .... char buffer[1001]; int len; while ((len = pBIO_read(bio, buffer, 1000)) > 0) { buffer[len] = 0; fprintf(file, buffer); } .... }
      
      

        
        
        
      

    
        
        
        
      
      

        
        
        
      

    
     

V618 इस तरह से 'fprintf' फ़ंक्शन को कॉल करना खतरनाक है, क्योंकि जो लाइन पास की जा रही है उसमें प्रारूप विनिर्देश हो सकते हैं। सुरक्षित कोड का उदाहरण: प्रिंटफ ("% s", str); asyncsslsocketlayer.cpp 2247



यदि, किसी फ़ाइल पर लिखते समय, नियंत्रण विनिर्देशक बफर में समाहित हैं, तो परिणाम अप्रत्याशित होगा। सुरक्षित तरीका:

 fprintf(file, "%s", buffer);
      
      

        
        
        
      

    
        
        
        
      
      

        
        
        
      

    
     

सामान्य तौर पर, इस त्रुटि को संभावित भेद्यता माना जा सकता है ।

चर 'गलती' के साथ कुछ गलत है

 static error_t client_send_propfind_request(....) { .... error_t err = 0; int code = 0; apr_hash_t * props = NULL; const char * target = path_uri_encode(remote_path, pool); char * url_path = apr_pstrdup(pool, target); WEBDAV_ERR(neon_get_props(&props, ras, url_path, NEON_DEPTH_ZERO, starting_props, false, pool)); if (err && (err == WEBDAV_ERR_DAV_REQUEST_FAILED)) .... }
      
      

        
        
        
      

    
        
        
        
      
      

        
        
        
      

    
     

चेतावनी: V560 सशर्त अभिव्यक्ति का एक हिस्सा हमेशा गलत होता है: (गलत == 1003)। webdavfilesystem.cpp 10990

निष्कर्ष

आप Embarcadero RAD Studio का उपयोग करने वाले प्रोग्रामर कहाँ हैं? एय! जैसा कि हमारे आंकड़े बताते हैं, लगभग कोई भी नहीं हैं। आओ और PVS- स्टूडियो कोड विश्लेषक की कोशिश करो!

यह लेख अंग्रेजी में है।

यदि आप इस लेख को अंग्रेजी बोलने वाले दर्शकों के साथ साझा करना चाहते हैं, तो कृपया अनुवाद के लिंक का उपयोग करें: एंड्री कारपोव। Embarcadero C ++ बिल्डर में विकसित ओपन-सोर्स प्रोजेक्ट WinSCP का एक चेक ।