https://bugzilla.novell.com/show_bug.cgi?id=248717 ------- Comment #3 from werner@novell.com 2007-02-27 11:32 MST ------- IMHO valgrind is not correct, the char array indstr[] in array_to_assign() is filled by the function inttostr() and this function provides a pointer on the result. Nevertheless, not only the pointer `is' but also the array `indstr[]' is initialised after inttostr(): array_to_assign (a, quoted) ARRAY *a; int quoted; { char *result, *valstr, *is; char indstr[INT_STRLEN_BOUND(intmax_t) + 1]; ARRAY_ELEMENT *ae; int rsize, rlen, elen; if (a == 0 || array_empty (a)) return((char *)NULL); result = (char *)xmalloc (rsize = 128); result[0] = '('; rlen = 1; for (ae = element_forw(a->head); ae != a->head; ae = element_forw(ae)) { is = inttostr (element_index(ae), indstr, sizeof(indstr)); valstr = element_value (ae) ? sh_double_quote (element_value(ae)) : (char *)NULL; elen = STRLEN (indstr) + 8 + STRLEN (valstr); [...] replacing STRLEN(indstr) with STRLEN(is) removes the message but nevertheless the line is correct even without the change. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.