क्यों() काम नहीं कर रहा है?

Question

मैं यूनिक्स में सी में प्रोग्रामिंग कर रहा हूं, और मैं कीबोर्ड से इनपुट पढ़ने के लिए gets का उपयोग कर रहा हूं।

warning: this program uses gets(), which is unsafe. 

किसी को भी मेरे कारण है कि यह हो रहा है बता सकते हैं: मैं हमेशा इस चेतावनी और कार्यक्रम चलना बंद हो जाते हैं?

Answer 1

जैसा कि पिछले उत्तरों में उल्लेख किया गया है के बजाय fgets का उपयोग करें।

लेकिन यह gets जैसा नहीं है, यह बिल्कुल काम नहीं करता है, यह बहुत ही असुरक्षित है। मेरा अनुमान है कि आपके पास आपके कोड में एक बग है जो fgets के साथ दिखाई देगा, इसलिए कृपया अपना स्रोत पोस्ट करें।

EDIT आपकी टिप्पणी में दी गई अद्यतन जानकारी के आधार पर मेरे पास कुछ सुझाव हैं।

• मैं आपकी मूल भाषा में एक अच्छा सी ट्यूटोरियल खोजने की सलाह देता हूं, Google आपका मित्र है। एक पुस्तक के रूप में मैं The C Programming Language

• यदि आपके पास नई जानकारी है तो उन्हें अपने मूल पोस्ट में संपादित करना एक अच्छा विचार है, खासकर यदि यह कोड है, तो यह लोगों के लिए समझने में आसान होगा कि आपका क्या मतलब है।

• आप एक स्ट्रिंग को मूल रूप से वर्णों की एक सरणी पढ़ने की कोशिश कर रहे हैं, जो कि एक ही चरित्र में है, जो निश्चित रूप से विफल हो जाएगा। आप जो करना चाहते हैं वह निम्न जैसा है।

  char username[256]; 
  char password[256]; 
  scanf("%s%s", username, password); 
  

  टिप्पणी करने के लिए/संपादन स्वतंत्र महसूस, मैं बुनियादी सी में भी बहुत जंग लगी हूँ

संपादित 2 jamesdlin के रूप में चेतावनी दी थी, scanf के उपयोग gets के रूप में के रूप में खतरनाक है।

Answer 2

gets असुरक्षित है क्योंकि आप इसे एक बफर देते हैं, लेकिन आप यह नहीं बताते कि बफर कितना बड़ा है। इनपुट बफर के अंत से पहले लिख सकता है, जो आपके कार्यक्रम को काफी शानदार तरीके से उड़ा सकता है। fgets उपयोग करने के बजाय एक सा बेहतर है क्योंकि आप यह कहना कितना बड़ा बफर है इस तरह, यह है:

const int bufsize = 4096; /* Or a #define or whatever */ 
char buffer[bufsize]; 

fgets(buffer, bufsize, stdin); 

... इसलिए प्रदान की आप इसे सही जानकारी देते हैं, यह बफर के अंत अतीत बारे में नहीं है और चीजों को उड़ाना।

थोड़ा OT, लेकिन:

आप बफर आकार के लिए एक const int उपयोग करने के लिए नहीं है, लेकिन मैं करूंगा दृढ़ता से सुझाव है कि आप सिर्फ दोनों स्थानों में एक शाब्दिक संख्या डाल नहीं है, क्योंकि अनिवार्य रूप से आप एक बदल जाएगा लेकिन बाद में नहीं। कंपाइलर मदद कर सकता है:

char buffer[4096]; 
fgets(buffer, (sizeof buffer/sizeof buffer[0]), stdin); 

वह अभिव्यक्ति संकलित समय पर संकलित हो जाती है, रनटाइम नहीं। यह टाइप करने के लिए एक दर्द है, तो मैं हेडर की मेरी हमेशा की तरह सेट में मैक्रो का उपयोग करने के लिए इस्तेमाल:

#define ARRAYCOUNT(a) (sizeof a/sizeof a[0]) 

... लेकिन मैं अपने शुद्ध सी के साथ पुराने हो चुके कुछ साल कर रहा हूँ, वहाँ शायद एक बेहतर तरीका है इन दिनों।

Answer 3

man gets का कहना है:

कभी का उपयोग हो जाता है()। क्योंकि यह असंभव अग्रिम कितने वर्ण हो जाता है() पढ़ा जाएगा में डेटा जानने के बिना बताने के लिए है, और क्योंकि हो जाता है() बफर के अंत अतीत पात्रों स्टोर करने के लिए जारी रहेगा, यह बहुत उपयोग करने के लिए खतरनाक है । यह का उपयोग कंप्यूटर सुरक्षा को तोड़ने के लिए किया गया है। इसके बजाए fgets() का प्रयोग करें।

Answer 4

gets() असुरक्षित है। यह एक पैरामीटर लेता है, एक चार बफर के लिए एक सूचक। खुद से पूछें कि आपको उस बफर को कितना बड़ा बनाना है और उपयोगकर्ता कितनी देर तक वापसी कुंजी को मारने के बिना इनपुट टाइप कर सकता है।

असल में, बफर ओवरफ्लो को() - fgets() का उपयोग करने से रोकने का कोई तरीका नहीं है।