Discussion:
fwd_ior threading race and patch
Felipe Magno de Almeida
2013-11-04 11:33:05 UTC
Permalink
Hello Karel,

We noticed a race with the fwd_ior member of Object when multiple
threads made calls concurrently. We've fixed it by moving the forward
object outside the reference and to the algorithms. The attached patch
has been working for over 1 year.

Any comments are appreciated,

Regards,
--
Felipe Magno de Almeida
Karel Gardas
2013-11-16 17:58:45 UTC
Permalink
Hello Felipe,

thanks a lot for your patch, but before merging it into source tree I'd
like to duplicate the issue myself. By any chance do you have as simple
as possible example which duplicates this issue? I'm asking for it since
from the very quick look into the patch I don't see how it fixes the
issue exactly -- so I'd like to get it here on my "desk" and investigate
a little bit this...

Thanks a lot,
Karel
Post by Felipe Magno de Almeida
Hello Karel,
We noticed a race with the fwd_ior member of Object when multiple
threads made calls concurrently. We've fixed it by moving the forward
object outside the reference and to the algorithms. The attached patch
has been working for over 1 year.
Any comments are appreciated,
Regards,
------------------------------------------------------------------------------
Android is increasing in popularity, but the open development platform that
developers love is also attractive to malware creators. Download this white
paper to learn more about secure code signing practices that can help keep
Android apps secure.
http://pubads.g.doubleclick.net/gampad/clk?id=65839951&iu=/4140/ostg.clktrk
_______________________________________________
Mico-devel mailing list
https://lists.sourceforge.net/lists/listinfo/mico-devel
--
Karel Gardas ***@objectsecurity.com
ObjectSecurity Ltd. http://www.objectsecurity.com
Felipe Magno de Almeida
2013-11-17 01:52:54 UTC
Permalink
Hello Karel,

I probably won't be able to give you a sample, but Ricardo Cosme
probably can. What I can do right now is say that this bug appears
with constant throwing of ForwardRequest, which causes the
modification of the reference in the fwd_ior member variable.

I can fill in more details later.

Regards,

On Sat, Nov 16, 2013 at 3:58 PM, Karel Gardas
Post by Karel Gardas
Hello Felipe,
thanks a lot for your patch, but before merging it into source tree I'd like
to duplicate the issue myself. By any chance do you have as simple as
possible example which duplicates this issue? I'm asking for it since from
the very quick look into the patch I don't see how it fixes the issue
exactly -- so I'd like to get it here on my "desk" and investigate a little
bit this...
Thanks a lot,
Karel
Post by Felipe Magno de Almeida
Hello Karel,
We noticed a race with the fwd_ior member of Object when multiple
threads made calls concurrently. We've fixed it by moving the forward
object outside the reference and to the algorithms. The attached patch
has been working for over 1 year.
Any comments are appreciated,
Regards,
------------------------------------------------------------------------------
Android is increasing in popularity, but the open development platform that
developers love is also attractive to malware creators. Download this white
paper to learn more about secure code signing practices that can help keep
Android apps secure.
http://pubads.g.doubleclick.net/gampad/clk?id=65839951&iu=/4140/ostg.clktrk
_______________________________________________
Mico-devel mailing list
https://lists.sourceforge.net/lists/listinfo/mico-devel
--
ObjectSecurity Ltd. http://www.objectsecurity.com
--
Felipe Magno de Almeida
Loading...