Re: clamav RWX

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 12 Jul 2010 07:48:29 -0400
Eric Paris <eparis@xxxxxxxxxxxxxx> wrote:

> 2010/7/12 Török Edwin <edwintorok@xxxxxxxxx>:
> 
> > [*]
> > I have some plans to make the JIT work without RWX, since ClamAV
> > has 2 phases:
> >  - load DB, JIT compile bytecode (should use only RW- mapping, but
> >   currently needs RWX)
> >  - execute (JIT compiled) bytecode (should change mapping to be R-X)
> 
> Just so you know that is going to require the same permissions.
> (Hopefully) The only way to get around the SELinux permissions is to
> have 2 separate mappings.  Basically in really really rough sudo-code,
> 
> file = open(filename, RWX);
> unlink(file);
> truncate(file, however big you need);
> exec_area = mmap(PROT_EXEC, file);
> write_area = mmap(PROT_WRITE, file);
> 
> then do all of the writing to write_area and all of the executing in
> exec_area.
> 
> http://people.redhat.com/drepper/selinux-mem.html

Yes I've seen that page, however that is not going to be so simple.
All the relocations are done assuming that the code will be executed
where you write it to (and all absolute jumps are generated the
same way).
Changing that would be a lot of work, and could introduce new bugs.

> 
> Simply using mremap to change a mapping from PROT_WRITE to PROT_EXEC
> will cause the same problems as just doing it at the same time.

Then I'd better write a small testcase before converting LLVM to do
that.
IIRC I tried something like that and worked, but I could have done
something wrong. Will try again to be sure.

Is there a boolean (other than execmem) that would allow RW -> RX
mprotect()?

Why is the 2 mappings approach more secure though?
What I want is generate some code at runtime, execute it, then maybe
change it again, and execute that. Both do just that.


Best regards,
--Edwin


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with
the words "unsubscribe selinux" without quotes as the message.


[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux