I recently used cppcheck on mg and I got this message:

[macro.c:41]: (error) Memory pointed to by 'lp1' is freed twice.

Looking at the code:

/* free lines allocated for string arguments */
if (maclhead != NULL) {
        for (lp1 = maclhead->l_fp; lp1 != maclhead; lp1 = lp2) {
                lp2 = lp1->l_fp;
                free(lp1);
        }
        free(lp1);
}

What do you guys think?



# Han
-- 
Please cc me, since I'm not subscribed.

Reply via email to