[MPlayer-dev-eng] [PATCH 1/4] String handling audit/cleanup

Alex Beregszaszi alex at rtfs.hu
Fri Mar 2 21:31:41 CET 2007


Hi,

As a general rule I would consider sending such patches in one diff file
for a directory.

in cpudetect.c:

-       sprintf(vendor,"%.4s%.4s%.4s",(char*)(regs+1),(char*)(regs
+3),(char*)(regs+2));
+       snprintf(vendor, sizeof vendor, "%.4s%.4s%.4s",(char*)(regs
+1),(char*)(regs+3),(char*)(regs+2));

change this to sizeof(vendor)

I see this is an issue in other files too.

Regarding unrarlib.c: please send it to upstream too!

--
Alex Beregszaszi




More information about the MPlayer-dev-eng mailing list