Please avoid typos in the patch subject. > To fix this possible bug, sfr->buf_size is assigned to a local variable, > and then this variable is checked before being used. How do you think about a wording variant like the following? Thus assign the data structure member “buf_size” to a local variable and check it before further usage. Would you like to add the tag “Fixes” to the commit message? Regards, Markus