Re: [PATCH 2/2] trace_uprobe: Simplify probes_seq_show()
From: Kees Cook
Date: Wed Mar 07 2018 - 17:17:24 EST
On Tue, Mar 6, 2018 at 8:12 PM, Ravi Bangoria
<ravi.bangoria@xxxxxxxxxxxxxxxxxx> wrote:
>
>
> On 03/07/2018 03:34 AM, Kees Cook wrote:
>> On Tue, Mar 6, 2018 at 12:12 AM, Ravi Bangoria
>> <ravi.bangoria@xxxxxxxxxxxxxxxxxx> wrote:
>>>
>>> On 02/08/2018 09:13 AM, Ravi Bangoria wrote:
>>>> Wang, ping :)
>>>>
>>>> Kees, I don't hear back from Wang and no one has reported any issues with
>>>> the patches yet. Can I have your Acked-by?
>> I didn't see a v2 of these patches with the output fixed?
>
> Kees, I didn't send v2 because those 0s seems unnecessary to me.
Oh, okay. I don't care either way. That's up to whoever consumes that
output. Usually it's bad form to change formatting for a userspace
API, but if nothing cares, then sure. :)
> Please let me know if you still wants to show them. Will re-spin :)
AIUI, it's "safer" to include them since that's what it did before.
-Kees
>
> Thanks,
> Ravi
>
> PS: Changing Masami's email address in cc.
>
--
Kees Cook
Pixel Security