home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #30 / NN_1992_30.iso / spool / comp / lang / c / 18470 < prev    next >
Encoding:
Text File  |  1992-12-16  |  4.3 KB  |  122 lines
  1. Newsgroups: comp.lang.c
  2. Path: sparky!uunet!ocsmd!ted
  3. From: ted@ocsmd.ocs.com (Ted Scott)
  4. Subject: Re: character pointers in funtion calls
  5. Message-ID: <BzD1w3.1z5@ocsmd.ocs.com>
  6. Sender: news@ocsmd.ocs.com
  7. Organization: Online Computer Systems, Inc.
  8. X-Newsreader: Tin 1.1 PL5
  9. References: <BzC5oG.HpB@ucunix.san.uc.edu>
  10. Date: Wed, 16 Dec 1992 16:31:13 GMT
  11. Lines: 109
  12.  
  13. Spandan Choudury (schoudhu@ucunix.san.uc.edu) wrote:
  14. : This may be a very simple question (I must be missing something I am sure,)
  15. Uh Huh.
  16. : but I really do not have the time to figure out to get rid of warnings as
  17. Assignment due soon?
  18. : happen with attempting to call a function to return and assign character
  19. : type pointer variables as in the example below appars with some system level 
  20.                                                              ^^^^^^^^^^^^^^^^
  21. Got yer drives backed up?  
  22. : C code that I am currently designing (actually in MINIX C on a 386, but I 
  23. : thought a C related newsgroup will be more appropriate to discuss this
  24. : matter in).
  25. : 
  26. : I am not looking for compiler options/flags to skip compilation warnigs.
  27. Ok.
  28. : I am looking only for code improvements.
  29. <sigh>
  30. : 
  31. : (Please mail.)
  32. <heavy sigh> Well, it *is* Christmas ;-) But I won't mail.
  33. : ..........................
  34. : 
  35.  
  36. Step 1: Get a good C primer and look at the memory sections, then walk
  37. through the code you present here and see if you can find some of these 
  38. problems:
  39.  
  40. : #include <stdio.h>       <- very good 
  41. : main()                   <- another useful construct, might want to 
  42.                               return an int though, maybe accept argc,argv
  43. : {
  44. :    char* c;              <- *THINK* about what you've allocated here
  45.                               (storage for a pointer)
  46.  
  47. :    c = f();              <- f doesn't return a char *, I'm not sure 
  48.                               if f returns anything but if it does its an
  49.                               int, also you probably get an implicit 
  50.                               declaration warning here?
  51.  
  52. :    printf ("%s\n", c);   <- If you get c to point to the right thing,
  53.                               this will work.
  54.  
  55. : }                        <- All in all a fine first attempt. You might 
  56.                               want to allocate some storage and pass it 
  57.                               into f.
  58. : 
  59. :  f()                     <- If you want this to return a char * then
  60.                               define it that way: char *f() 
  61. : {
  62. :   char* i;               <- Look up "automatic variables" in the primer 
  63.                               you bought in step 1. Notice the part about 
  64.                               the storage becomming undefined when the 
  65.                               function exits.
  66.  
  67. :   i = "phoo";            <- A dangerous practice in the real world if
  68.                               you can't guarantee that *i will never be
  69.                               attempted to be modified.
  70.  
  71. :   return (i);            <- (and implicitly free the memory on the stack
  72.                               where i had a happy albiet brief life) 
  73.                               returning the old value of i which may or may 
  74.                               not be overwritten when printf uses the stack. 
  75. : }
  76.  
  77.  
  78. Try something like:
  79.  
  80. #include <stdio.h>
  81.  
  82. char  *phideaux(char *foo, int size) {
  83.  
  84.  
  85.    if (size < 5) return (NULL);
  86.    
  87.    if (!foo) if ((foo = (char *) malloc(size)) == NULL)  exit(1);
  88.    /* people who write code like this should be taken behind the woodshed... */
  89.  
  90.    memcpy(foo,"phoo",5);
  91.    return(foo);
  92. }
  93.  
  94. int main(int argc,char *argv) {
  95.    
  96.     char *c;
  97.  
  98.     if ((c = phideaux(c,4)) != NULL) {
  99.        printf("%s\n",c);
  100.        return(0);
  101.     }
  102.     
  103.     fprintf(stderr,"phideaux is a dog function that contains holes "
  104.                    "big enough to drive a Mack (tm) truck through!\n"
  105.                    "and therefore functions that manipulate storage "
  106.                    "should be used within the context that they were "
  107.                    "designed for and should not be violated. Otherwise "
  108.                    "they have every right to overwrite your inode table "
  109.                    "with a verbose error message acknowledging the "
  110.                    "\"splendorifious stupiditity\" of the implementor!");
  111.  
  112.    return (1);
  113. }
  114.  
  115.  
  116.     
  117. --
  118.  
  119. -Ted Scott          
  120. tscott@ocsmd.ocs.com           I was told that I'm a P.C. person:
  121. (301) 601-2252                               Politically Challenged, that is.
  122.