Re: [PATCH v5 1/5] perf metric: Event "Compat" value supports matching multiple identifiers

From: Jing Zhang
Date: Wed Aug 02 2023 - 05:47:43 EST




在 2023/8/2 下午5:43, John Garry 写道:
> On 02/08/2023 10:38, Jing Zhang wrote:
>>>>> n;
>>>>> +
>>>>> +    str = strdup(compat);
>>>> why duplicate this? are you modifying something?
>>>>
>>> This is really a redundant step, I will remove it.
>>>
>> Hi John,
>>
>> I reviewed this code again and found that it still needs to duplicate "compat" because "compat" is a
>> const str* type and cannot be used as a parameter for the strtok_r function. If it is cast to char*,
>> using "compat" as a parameter for strtok_r is also unsafe and can cause a "Segmentation fault" error.
>> Therefore, let's keep the step of duplicating "compat".
>
> ok, so then please add a small comment on why the strdup() call is needed.
>

No problem.

Thanks,
Jing