Messages in this thread | | | Date | Sat, 20 Jan 2024 15:45:11 +0100 | Subject | Re: [PATCH v3] x86/xen: Add some null pointer checking to smp.c | From | Markus Elfring <> |
| |
> kasprintf() returns a pointer to dynamically allocated memory > which can be NULL upon failure. Ensure the allocation was successful > by checking the pointer validity.
How do you think about to refer to the function name instead of the file name in the patch subject?
… > +++ b/arch/x86/xen/smp.c … > @@ -114,6 +124,8 @@ int xen_smp_intr_init(unsigned int cpu) > > return 0; > > + fail_mem: > + rc = -ENOMEM; > fail: > xen_smp_intr_free(cpu); > return rc;
Is it currently preferred to start labels in the first text column?
Regards, Markus
| |