Re: fs/seq_file: Replace a seq_printf() call by seq_puts() in seq_hex_dump()

From: Markus Elfring
Date: Wed Jul 03 2019 - 07:01:43 EST


> Looks good,

Thanks for your feedback.


> but have you checked whether "m" could ever be NULL

I wonder about this enquiry.

This function parameter should be valid as usual.
Thus it should not be a null pointer under ordinary conditions.


> and whether seq_puts() has a check for that ?

These output functions do not provide an explicit sanity check for their first parameter
so far.

Regards,
Markus