On 5 Jan 2008, at 13:46, Archana wrote:
> Hello ,
> I had previously send the fix for this bug, but i cant find it now,
> so I am sending it again.
> if ((fp = ctf_bufopen(&ctfsect, &symsect, &strsect, error)) == NULL)
> + dprintf(?cannot open ctf_bufopen()?);
> return (NULL);
Even without looking at the bug report or the relevant source file,
it's obvious by inspection that this cannot be the correct fix, nor
can it have been tested.
And in fact bug 6309693 relates to locale definitions, so it's rather
unlikely that this patch to what looks like a kernel source file will
Judging from the number of similar postings, I'm guessing that some
professor at the Amrita School of Engineering has set "Find and fix a
Solaris bug" as a piece of coursework for an undergraduate course.
Now whilst it's good for Solaris' outreach to involve these students,
and good for their education, I have to question the value of the
exercise if the students don't take the time or care to develop
correct solutions and submit them in a useful way... It just ends up
offloading their education onto the people who sponsor the fixes.