🚀 go-pugleaf

* VERSION: g++ version 2.1

* SOURCE FILE: collect2.c (from the distribution)

    =======================================================================
    ORIGINAL VERSION (savestring() #line 393)
        /* Make a copy of a string INPUT with size SIZE.  */

        char *
        savestring (input, size)
             char *input;
             int size;
        {
          char *output = (char *) xmalloc (size + 1);
-         strcpy (output, input);
          return output;
        }


    CORRECTED VERSION (savestring() #line 393)
        /* Make a copy of a string INPUT with size SIZE.  */

        char *
        savestring (input, size)
             char *input;
             int size;
        {
          char *output = (char *) xmalloc (size + 1);
+         strncpy (output, input, size); /* no NULL terminator in input */
+         output[size]=0;
          return output;
        }
    =======================================================================

* DESCRIPTION OF INCORRECT BEHAVIOR:

    Collect2 overwrites the heap and generates a segment violation
    when a -f.. or -m... switch is used because input does not have
    a NULL terminator and too many bytes are copied.  The fix is
    shown above.