From: Steve Lawrence <slawrence@tresys.com>
To: Stephen Smalley <sds@tycho.nsa.gov>,
Sven Vermeulen <sven.vermeulen@siphos.be>
Cc: SELinux <selinux@tycho.nsa.gov>
Subject: Re: SELinux Userspace Release: 20140826-rc1
Date: Fri, 29 Aug 2014 10:22:11 -0400 [thread overview]
Message-ID: <54008C93.5060300@tresys.com> (raw)
In-Reply-To: <54008AC4.6080600@tycho.nsa.gov>
On 08/29/2014 10:14 AM, Stephen Smalley wrote:
> On 08/29/2014 10:00 AM, Sven Vermeulen wrote:
>> On Fri, Aug 29, 2014 at 2:56 PM, Steve Lawrence <slawrence@tresys.com> wrote:
>>>>>>> Segmentation fault
>>>>> [...]
>>>>>> Can you provide a copy of your original policy prior to conversion?
>>>>>
>>>>> If you mean the policy.29 file, certainly. You can wget it from
>>>>> http://dev.gentoo.org/~swift/tmp/20140828-policy.29
>>>>
>>>> No, the contents of /etc/selinux/mcs. The migration script converts the
>>>> old policy module store, not the final kernel policy file.
>>>>
>>>
>>> Hmm, I'm unable to reproduce this. I think the policy store that Stephen
>>> mentions will be help to reproduce it.
>>>
>>
>> Certainly.
>>
>> The policy store can be found at
>> http://dev.gentoo.org/~swift/tmp/20140829-etc-selinux-mcs.tar.gz
>
> Hmm...semanage_migrate_store worked for me on that policy store.
>
> Can you reproduce the fault? If so, can you get debug info?
> Build with debug flags and run semanage_migrate_store under valgrind,
> perhaps?
>
We are able to get the segfault and have gotten a backtrace. It looks
like it has to do with how optionals are handled and how we reset state
when an optional is disabled. The fix in this particular case is pretty
simple, but I think we need to go through the rest of the reset state
code and ensure we aren't making similar mistakes. Might take a little
bit of time.
next prev parent reply other threads:[~2014-08-29 14:22 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-26 15:31 SELinux Userspace Release: 20140826-rc1 Steve Lawrence
2014-08-27 15:06 ` Sven Vermeulen
2014-08-27 16:10 ` Steve Lawrence
2014-08-28 19:58 ` Sven Vermeulen
2014-08-28 20:15 ` Stephen Smalley
2014-08-28 20:52 ` Sven Vermeulen
2014-08-29 12:11 ` Stephen Smalley
2014-08-29 12:56 ` Steve Lawrence
2014-08-29 14:00 ` Sven Vermeulen
2014-08-29 14:14 ` Stephen Smalley
2014-08-29 14:18 ` Stephen Smalley
2014-08-29 14:22 ` Steve Lawrence [this message]
2014-09-14 9:31 ` Sven Vermeulen
2014-09-15 12:12 ` Steve Lawrence
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=54008C93.5060300@tresys.com \
--to=slawrence@tresys.com \
--cc=sds@tycho.nsa.gov \
--cc=selinux@tycho.nsa.gov \
--cc=sven.vermeulen@siphos.be \
/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.