| Message ID | 20250408150354.104532-1-bsdhenrymartin@gmail.com |
|---|---|
| Headers | show |
| Series | cpufreq: scmi/scpi: Fix NULL pointer dereference in get_rate() | expand |
On Wed, Apr 09, 2025 at 01:48:33PM +0200, Markus Elfring wrote: > >> Can any other summary phrase variants become more desirable accordingly? I agree with Sudeep, the above sentence is completely incomprehensible to me > > > > This is meaningless, sorry can't parse. Ignoring it as others in the > > community are doing already. > Do you care if the term “null pointer dereference” would be used in consistent ways? > ...this is more comprehensible, but again I cannot grasp what's yor advice specifically on this commit message. Thanks, Cristian
>>>> Can any other summary phrase variants become more desirable accordingly? > > I agree with Sudeep, the above sentence is completely incomprehensible > to me Can any suggestions gain acceptance also for better summary phrases? >>> This is meaningless, sorry can't parse. Ignoring it as others in the >>> community are doing already. >> Do you care if the term “null pointer dereference” would be used in consistent ways? > > ...this is more comprehensible, Thanks for another bit of constructive information. > but again I cannot grasp what's yor advice > specifically on this commit message. May the usage of abbreviations be reconsidered once more also for such messages (in presented update steps)? Regards, Markus
On Wed, Apr 09, 2025 at 02:25:52PM +0200, Markus Elfring wrote: > >>>> Can any other summary phrase variants become more desirable accordingly? > > > > I agree with Sudeep, the above sentence is completely incomprehensible > > to me > > Can any suggestions gain acceptance also for better summary phrases? > > > > >>> This is meaningless, sorry can't parse. Ignoring it as others in the > >>> community are doing already. > >> Do you care if the term “null pointer dereference” would be used in consistent ways? > > > > ...this is more comprehensible, > > Thanks for another bit of constructive information. > > > > but again I cannot grasp what's yor advice > > specifically on this commit message. > May the usage of abbreviations be reconsidered once more also for such messages > (in presented update steps)? > Still can't understand you. Sorry for that. Alternatively, you can do what I sometimes do: just write the whole commit log as you would expect and see if that helps. I am sure that helps, so please do that.
>> May the usage of abbreviations be reconsidered once more also for such messages >> (in presented update steps)? > > Still can't understand you. Sorry for that. … Will any communication challenges need further clarifications also according to wordings like the following? * null-ptr-deref * null pointer dereference Regards, Markus
On 08-04-25, 23:03, Henry Martin wrote: > This series fixes potential NULL pointer dereferences in scmi_cpufreq_get_rate() > and scpi_cpufreq_get_rate() when cpufreq_cpu_get_raw() returns NULL. > > Henry Martin (2): > cpufreq: scmi: Fix null-ptr-deref in scmi_cpufreq_get_rate() > cpufreq: scpi: Fix null-ptr-deref in scpi_cpufreq_get_rate() > > drivers/cpufreq/scmi-cpufreq.c | 10 ++++++++-- > drivers/cpufreq/scpi-cpufreq.c | 13 ++++++++++--- > 2 files changed, 18 insertions(+), 5 deletions(-) Applied. Thanks.