Re: TLSF memory corruption detection

Willy Lambert <[email protected]> Tue, 15 Sep 2015 15:56:45 +0200
Newsgroups gmane.science.robotics.orocos.devel
Message-ID <CAKvQZ_2bRQ08MJHofR-0vkH0RD715F11tQ9txoT3xU7xfZ48qQ@mail.gmail.com>
--===============1517489790==
Content-Type: multipart/alternative; boundary=089e0158c95a6ec56d051fc98dbd

--089e0158c95a6ec56d051fc98dbd
Content-Type: text/plain; charset=UTF-8

This is definitely a good thing.

2015-09-15 13:53 GMT+02:00 Ruben Smits <[email protected]>:

> Hi all,
>
> There is currently a PR waiting for merge:
> https://github.com/orocos-toolchain/rtt/pull/77
> The PR adds functionality to the TLSF memory pool to detect memory
> corruption, which can happen due to double frees or non-thread safe
> handling of rt allocated variables.
>
> Previously applications would just continue to work with a corrupted TLSF
> pool resulting in undefined behaviour.  With these patches the application
> would abort,  since the TLSF memory pool is corrupted (we cannot detect it
> beforehand, only after the corruption has taken place) there is no real
> sane thing to do from that point and the abort is similar to a segfault.
>
> Before we merge this PR I would like everyone to fully understand the
> consequences of it as it might surface tlsf memory corruption issues with
> your applications that were undetected before.
>
> If you have big objections please speak up.
>
> Regards,
>
> Ruben
> --
> Ruben Smits, Roboticist - Founder
> +32 479 511 786
> Intermodalics - Kapeldreef 60, 3001 Heverlee - BELGIUM
> www.intermodalics.eu
>
> --
> Orocos-Dev mailing list
> [email protected]
> http://lists.mech.kuleuven.be/mailman/listinfo/orocos-dev
>
>

--089e0158c95a6ec56d051fc98dbd
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">This is definitely a good thing. <br></div><div class=3D"g=
mail_extra"><br><div class=3D"gmail_quote">2015-09-15 13:53 GMT+02:00 Ruben=
 Smits <span dir=3D"ltr">&lt;<a href=3D"mailto:[email protected]=
" target=3D"_blank">[email protected]</a>&gt;</span>:<br><blockq=
uote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc =
solid;padding-left:1ex"><div dir=3D"ltr">Hi all,<div><br></div><div>There i=
s currently a PR waiting for merge:=C2=A0<a href=3D"https://github.com/oroc=
os-toolchain/rtt/pull/77" target=3D"_blank">https://github.com/orocos-toolc=
hain/rtt/pull/77</a></div><div>The PR adds functionality to the TLSF memory=
 pool to detect memory corruption, which can happen due to double frees or =
non-thread safe handling of rt allocated variables.</div><div><br></div><di=
v>Previously applications would just continue to work with a corrupted TLSF=
 pool resulting in undefined behaviour.=C2=A0 With these patches the applic=
ation would abort, =C2=A0since the TLSF memory pool is corrupted (we cannot=
 detect it beforehand, only after the corruption has taken place) there is =
no real sane thing to do from that point and the abort is similar to a segf=
ault.</div><div><br></div><div>Before we merge this PR I would like everyon=
e to fully understand the consequences of it as it might surface tlsf memor=
y corruption issues with your applications that were undetected before.</di=
v><div><br></div><div>If you have big objections please speak up.</div><div=
><br></div><div>Regards,</div><div><br></div><div>Ruben</div></div><span cl=
ass=3D"HOEnZb"><font color=3D"#888888"><div dir=3D"ltr">-- <br></div><div d=
ir=3D"ltr"><div><div>Ruben Smits, Roboticist - Founder</div><div>+32 479 51=
1 786</div><div>Intermodalics - Kapeldreef 60, 3001 Heverlee - BELGIUM</div=
><div><a href=3D"http://www.intermodalics.eu" target=3D"_blank">www.intermo=
dalics.eu</a></div></div></div>
</font></span><br>--<br>
Orocos-Dev mailing list<br>
<a href=3D"mailto:[email protected]">[email protected].=
kuleuven.be</a><br>
<a href=3D"http://lists.mech.kuleuven.be/mailman/listinfo/orocos-dev" rel=
=3D"noreferrer" target=3D"_blank">http://lists.mech.kuleuven.be/mailman/lis=
tinfo/orocos-dev</a><br>
<br></blockquote></div><br></div>

--089e0158c95a6ec56d051fc98dbd--

--===============1517489790==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

-- 
Orocos-Dev mailing list
[email protected]
http://lists.mech.kuleuven.be/mailman/listinfo/orocos-dev

--===============1517489790==--