On Fri, Jun 28, 2024 at 02:54:53PM +0300, Vladimir 'phcoder' Serbinenko wrote:
> > +  if (protector == NULL || protector->name == NULL || grub_strlen 
> > (protector->name) == 0)
> > +    return GRUB_ERR_BAD_ARGUMENT;
> > +
> Here and in the other places you miss grub_error. Note that the
> message in such technical cases should be left untranslated (no N_
> mark).
> 
Ok. I'll add grub_error to provide the better error messages.

> > +  if (protector == NULL || grub_strlen (protector) == 0)
> 
> Rather than checking strlen just do protector[0] == '\0'

Ah, right. It's more efficient.

Gary Lin

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

Reply via email to