All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Kramer, Zach" <Zach.Kramer@cognex.com>
To: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Cc: lttng-dev <lttng-dev@lists.lttng.org>
Subject: Re: [EXTERNAL] Re: Problem with application changing UID
Date: Mon, 7 Oct 2019 13:25:51 +0000	[thread overview]
Message-ID: <SN4PR0601MB361478F2CC4246282CEC158D919B0__9551.17312712304$1570454775$gmane$org@SN4PR0601MB3614.namprd06.prod.outlook.com> (raw)
In-Reply-To: <1505194855.4263.1569873368480.JavaMail.zimbra@efficios.com>

Thanks for the tips. The stopping/starting issue (resulting in no more events) still exists with a live session, but it is related to Trace Compass reading the logs, not LTTng writing them. The logs are still written to and Babeltrace can view them fine, but Trace Compass cannot view past the point where the session was stopped. Thus, this is now outside the realm of this mailing list.

Best,
Zach

-----Original Message-----
From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com> 
Sent: Monday, September 30, 2019 9:56 PM
To: Kramer, Zach <Zach.Kramer@cognex.com>
Cc: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>; lttng-dev <lttng-dev@lists.lttng.org>
Subject: Re: [lttng-dev] [EXTERNAL] Re: Problem with application changing UID

----- On Sep 27, 2019, at 10:07 AM, Kramer, Zach Zach.Kramer@cognex.com wrote:

> Hi,
>
> Following up on this.
>
> Unfortunately I was not able to test the patch because our build 
> system relies on using the lttng version provided by poky -- updating 
> is not something we can easily do.
>
> I seemingly solved this problem in another manner, though. I built the 
> tracepoint provider package into a shared object and dlopen() it after 
> the root privileges have been dropped. This seems to have solved the 
> UID problem, because the app only registers itself under the UID that it will trace under.
>
> This resulted in another problem, which is that the metadata does not 
> exist until the session is stopped,

This is expected.

> and once the session is started again it is no longer tracing.

Well that means your solution does not completely work.

> In the lttng documentation I found:
>
> "Important: You need to stop tracing to make LTTng flush the remaining 
> trace data and make the trace readable."
>
> I interpreted this as "the metadata will not exist until the session 
> is stopped." Is this correct?
>
> Regardless, I enabled the channel 'metadata' and configured it to 
> flush via the switch timer, which then gives my traces metadata.
>
> My problem thus is seemingly solved. Do you see any flaws on the lttng 
> side of this approach?

If you want to consume your traces without stopping tracing, you need to use the lttng "live" mode or the new "session rotation" feature introduced in lttng 2.11.

Without those features, reading a trace while it's being produced is racy and may yield to corrupted traces.

Thanks,

Mathieu


>
> Best,
> Zach
>
> -----Original Message-----
> From: lttng-dev <lttng-dev-bounces@lists.lttng.org> On Behalf Of 
> Kramer, Zach
> Sent: Tuesday, September 24, 2019 6:01 PM
> To: Jonathan Rajotte-Julien <jonathan.rajotte-julien@efficios.com>
> Cc: lttng-dev@lists.lttng.org
> Subject: Re: [lttng-dev] [EXTERNAL] Re: Problem with application 
> changing UID
>
> Hi,
>
> Thanks for the quick response! This looks interesting -- I will 
> experiment with it and come back with any results.
>
> Cheers,
> Zach
>
>
> -----Original Message-----
> From: Jonathan Rajotte-Julien <jonathan.rajotte-julien@efficios.com>
> Sent: Tuesday, September 24, 2019 5:08 PM
> To: Kramer, Zach <Zach.Kramer@cognex.com>
> Cc: lttng-dev@lists.lttng.org
> Subject: [EXTERNAL] Re: [lttng-dev] Problem with application changing 
> UID
>
> Hi Zach,
>
> Thanks for reaching out.
>
> lttng-ust does not support the change of uid once the application is 
> registered to the lttng-sessiond daemon. I think that we use the uid 
> received on registration for all subsequent operations.
>
> Gabriel Pollo-Guilbert actually worked on this this summer. You can 
> check out the proposed wrapper for setuid here [1]. You will need to 
> LD_PRELOAD it on the start of you application. It basically unregister 
> the application and re-register it.
>
> [1]
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Flist
> s.lttng.org%2Fpipermail%2Flttng-dev%2F2019-June%2F029035.html&amp;data
> =02%7C01%7CZach.Kramer%40cognex.com%7C0a0ab1502ae24e4a4a9708d745e03d3a
> %7Cc12007a4882b4381b05ab783431570c7%7C1%7C0%7C637054701722761406&amp;s
> data=%2FQCW1bbnLXEPcs1PQKKoa1v9D5ZSQSiGBwbO6khwhYI%3D&amp;reserved=0
> This should be applied on master of lttng-ust.
> Make sure to use lttng-tools master also. Same for lttng-modules if necessary.
>
> Would you be interested in giving it a try?
>
> Cheers
>
> On Tue, Sep 24, 2019 at 02:32:41PM +0000, Kramer, Zach wrote:
>> Hi,
>>
>> Is LTTng intended to support userspace applications that change their 
>> UID at run-time? As in, is there an expected behavior for when this happens?
>>
>> For example:
>>
>>   1.  Embedded device boots
>>   2.  My daemon is launched as root via systemd
>>   3.  Runs privileged code
>>   4.  Changes UID to a less privileged user (500)
>>   5.  Creates LTTng session
>>      *   If session already exists, destroy it first
>>   6.  <if ‘systemctl stop’ is called>: Destroy session
>>      *   Otherwise it will be destroyed next daemon launch in step 5
>>
>> This causes many conflicts with the trace folders that are created. 
>> Most of the time, LTTng creates a folder + metadata for both root and 
>> the user, then puts traces in the user folder. Other times, it may 
>> create a folder just for the user. This is seemingly random, since 
>> it’s a fresh device boot each time. If the daemon is launched 
>> directly (i.e. not from systemd), then the root folder gets the 
>> traces and the user folder gets the metadata. Here is a more detailed
>> explanation:
>>
>>
>> Case
>> Command
>> Result
>>
>> Comments
>> 1
>> [Device boot]
>> systemctl start daemon
>> …../uid/500/32-bit --> has metadata and trace logs
>>
>> Most times, this also happens:
>> …../uid/0/32-bit --> has metadata but no trace logs
>>
>> [cid:image001.png@01D56F9D.AF158770]
>> Occasionally, the uid/0 folder is not created at all. This seems 
>> random, since this is tested by rebooting the device several times.
>> 2
>> [Device boot]
>> systemctl start daemon
>> systemctl stop daemon
>>
>>   *   This uses LTTng C-API to destroy session
>> …../uid/500/32-bit --> has metadata but the logs were cleared
>>
>> Most times, this also happens:
>> …../uid/0/32-bit --> has metadata but no trace logs
>>
>> [cid:image001.png@01D56F9D.AF158770]
>> The logs are cleared when ‘lttng destroy sess’ is called via the LTTng C-API.
>> From my understanding, this should not happen.
>>
>> Destroying the daemon from command line behaves properly. From my 
>> understanding, this should be practically the exact same command.
>> 3
>> root@device: daemon
>>
>> (launching via command-line)
>> When daemon is killed or session is stopped: mimics case 1
>>
>> When daemon is alive:
>> …./uid/0/32-bit --> has trace logs but empty metadata
>>
>> …./uid/500/32-bit --> has metadata but empty trace logs 
>> [cid:image002.png@01D56F9D.AF158770]
>>
>>
>> Is this use-case supported?
>>
>> Unfortunately, the logs are huge and contain sensitive information. 
>> If they can help a substantial amount, I can prune them.
>>
>> Thanks and best regards,
>> Zach
>
>
>
>
>> _______________________________________________
>> lttng-dev mailing list
>> lttng-dev@lists.lttng.org
>> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Flis
>> t
>> s.lttng.org%2Fcgi-bin%2Fmailman%2Flistinfo%2Flttng-dev&amp;data=02%7C
>> 0
>> 1%7CZach.Kramer%40cognex.com%7C0fa610945b6a4857ce0408d74100fa70%7Cc12
>> 0 
>> 07a4882b4381b05ab783431570c7%7C1%7C0%7C637049344808702442&amp;sdata=m
>> A
>> uYCBhxqSdNuYSecmk4FmlaujH8yEX7HGEi0Ysw4ko%3D&amp;reserved=0
>
>
> --
> Jonathan Rajotte-Julien
> EfficiOS
> _______________________________________________
> lttng-dev mailing list
> lttng-dev@lists.lttng.org
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Flist
> s.lttng.org%2Fcgi-bin%2Fmailman%2Flistinfo%2Flttng-dev&amp;data=02%7C0
> 1%7CZach.Kramer%40cognex.com%7C0a0ab1502ae24e4a4a9708d745e03d3a%7Cc120
> 07a4882b4381b05ab783431570c7%7C1%7C0%7C637054701722761406&amp;sdata=ZN
> 1%2FNSV2Gsxx88tbkSrQFn5LR0Kyc%2FfidPKTkPgP4Ag%3D&amp;reserved=0
>
> _______________________________________________
> lttng-dev mailing list
> lttng-dev@lists.lttng.org
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Flist
> s.lttng.org%2Fcgi-bin%2Fmailman%2Flistinfo%2Flttng-dev&amp;data=02%7C0
> 1%7CZach.Kramer%40cognex.com%7C0a0ab1502ae24e4a4a9708d745e03d3a%7Cc120
> 07a4882b4381b05ab783431570c7%7C1%7C0%7C637054701722761406&amp;sdata=ZN
> 1%2FNSV2Gsxx88tbkSrQFn5LR0Kyc%2FfidPKTkPgP4Ag%3D&amp;reserved=0

--
Mathieu Desnoyers
EfficiOS Inc.
https://nam04.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.efficios.com&amp;data=02%7C01%7CZach.Kramer%40cognex.com%7C0a0ab1502ae24e4a4a9708d745e03d3a%7Cc12007a4882b4381b05ab783431570c7%7C1%7C0%7C637054701722761406&amp;sdata=s3hl9M6zXFQQQD4VOGlAUe4U6E5ecZHCerTeQIfq%2FEk%3D&amp;reserved=0

_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

      parent reply	other threads:[~2019-10-07 13:25 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <SN4PR0601MB3614967ABCE12621B316C89D91840@SN4PR0601MB3614.namprd06.prod.outlook.com>
2019-09-24 15:07 ` Problem with application changing UID Jonathan Rajotte-Julien
2019-09-24 15:15 ` Jonathan Rajotte-Julien
     [not found] ` <20190924151515.GH187569@joraj-alpa>
2019-09-24 15:52   ` [EXTERNAL] " Kramer, Zach
     [not found] ` <20190924150754.GG187569@joraj-alpa>
2019-09-24 16:01   ` Kramer, Zach
     [not found]   ` <SN4PR0601MB36143DD719CB75559F9ABDD891840@SN4PR0601MB3614.namprd06.prod.outlook.com>
2019-09-24 16:06     ` Jonathan Rajotte-Julien
     [not found]     ` <20190924160604.GK187569@joraj-alpa>
2019-09-24 16:13       ` Kramer, Zach
     [not found]       ` <SN4PR0601MB361479BD3396866D29AA39F791840@SN4PR0601MB3614.namprd06.prod.outlook.com>
2019-09-24 18:45         ` Jonathan Rajotte-Julien
2019-09-27 14:07     ` Kramer, Zach
     [not found]     ` <SN4PR0601MB36145C3D44B796739CD7AF0691810@SN4PR0601MB3614.namprd06.prod.outlook.com>
2019-09-30 19:56       ` Mathieu Desnoyers
     [not found]       ` <1505194855.4263.1569873368480.JavaMail.zimbra@efficios.com>
2019-10-07 13:25         ` Kramer, Zach [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='SN4PR0601MB361478F2CC4246282CEC158D919B0__9551.17312712304$1570454775$gmane$org@SN4PR0601MB3614.namprd06.prod.outlook.com' \
    --to=zach.kramer@cognex.com \
    --cc=lttng-dev@lists.lttng.org \
    --cc=mathieu.desnoyers@efficios.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.