On Sun, Aug 31, 2008 at 07:23:41PM +0200, Felix Zielcke wrote:
> +       *uuid = grub_malloc (sizeof ("YYYY-MM-DD-HH-mm-ss-hh")

Lovely :-)

> + sizeof ('\0'));

Is this really needed?  sizeof("foo") implicitly includes the trailing \0.

-- 
Robert Millan

  The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and
  how) you may access your data; but nobody's threatening your freedom: we
  still allow you to remove your data and not access it at all."


_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to