שלום,

אני צריך לממש תוכנית של מעין "שורת פקודה" שכזו, כך שעבור כל שורה שאני קולט (בלולאת while) צריכה לעשות משהו.
עכשיו, משום מה אחרי כמה שורות של קליטת שורות מהמשתמש, התוכנית עפה לי ומופיעה לי השגיאה הבאה:



ניסיתי להבין מאיפה הבעיה, לא הצלחתי.
הנה הקוד, אשמח אם מישהו יזרוק עצם:

main:

קוד PHP:

 int main() {
    char *line;
    line = readLine();
    while(strcmp(line, "!quit")) {
        free(line);
        line = readLine();
    }
    free(line);
} 


(חשדתי שהבעיה היא עם שחרור הזיכרון שאני עושה, אבל גם אם אני מנסה להוריד את השורה הזו, הבעיה נותרת. מה גם שהגיונית לא אמורה להיות עם זה בעיה.)

הפונקציה readLine:

קוד PHP:

 char *readLine() {
    char *line;
    int phsize, lsize, c;
    phsize = 1;
    lsize = 0;
    line = (char *) malloc((1 + phsize) * sizeof(char));
    if(!line) {
        printf("Error allocating memory. exiting...");
        exit (1);
    }
    while( (c = getchar()) != '\n') {
        if(lsize == phsize) {
            (char *) realloc(line, (1 + phsize*2)*sizeof(char));
            if(!line) {
                printf("Error allocating memory. exiting...");
                exit (1);
            }
            phsize *= 2;
        }
        line[lsize++] = c;
    }
    (char *)realloc(line, (1 + lsize)*sizeof(char));
    line[lsize] = '\0';
    return line;
} 


תודה!

אוקיי, הבנתי מה הבעיה.
שכחתי ש-realloc, במידה והיא מעתיקה את הנתונים למיקום חדש, מחזירה מצביע למיקום החדש בזיכרון, ולא עשיתי השמה ל-line של המיקום החדש, מה שגרם להחזרת מצביע למיקום שפּוּנה כבר..

בקיצור, התיקון צריך להיות כזה:
בתוך הלולאה:

קוד PHP:

 line = (char *) realloc(line, (1 + phsize*2)*sizeof(char)); 


מחוץ ללולאה:

קוד PHP:

 line = (char *)realloc(line, (1 + lsize)*sizeof(char)); 


אתה משתמש ב-realloc לא נכון. אתה מתעלם מערך החזרה שלו.
http://www.cplusplus.com/reference/...cstdlib/realloc

(קרדיט למרשי)
אמר לה ינאי מלכא לדביתיה אל תתיראי מן הפרושין ולא ממי שאינן פרושין אלא מן הצבועין שדומין לפרושין שמעשיהן כמעשה זמרי ומבקשין שכר כפנחס
אמר פסטן: שניהם גרועים, אבל עדיף להיות טיפש מאשר שקרן.

תודה

אגב, ראיתי בכמה מקומות שלא מקובל להשתמש ב-casting על realloc ו-malloc, כי אתה בעצם מגביל את עצמך ואתה פוגע ב"גמישות", מה שיוצר יותר סיכויים לבאגים בעתיד. אתה מכיר משהו כזה?

אין לי הרבה ניסיון בתכנות ב-C, אבל לא ברור לי ככה בשלוף איזה באג זה עלול ליצור. (אלא אם כן אתה מדבר על איזו תופעה הזויה כתוצאה משרשור של המון קאסטים, או משהו מוזר כזה.)

השאלה היא למה בכלל צריך לשים את הקאסט הזה?
ב-CPP אתה חייב, כי אין קאסט אוטומטי בין הפויינטרים: http://ideone.com/clone/6xQks
אבל ב-C יש: http://ideone.com/HVybU ולא ברור מה הקאסט מוסיף. במקרים אחרים, אפשר להגיד שהקאסט עוזר לשים לב שמתבצעת המרה בין טיפוסים שונים, אבל כאן יש malloc - מובן מאליו שאיפשהו נעשה קאסט, כי אי-אפשר להשתמש ב-void*‎ ישירות...

(כל האמור לגבי C90 בלבד.)

(קרדיט למרשי)
אמר לה ינאי מלכא לדביתיה אל תתיראי מן הפרושין ולא ממי שאינן פרושין אלא מן הצבועין שדומין לפרושין שמעשיהן כמעשה זמרי ומבקשין שכר כפנחס
אמר פסטן: שניהם גרועים, אבל עדיף להיות טיפש מאשר שקרן.

סבבה, הבנתי אותך - תודה.