weixin_39668898
weixin_39668898
2020-12-26 07:05

Try to fix NodeTest race condition

YARP_serversql/NameServiceOnTriples: Protect concurrent access to internal variables using a mutex

Fixes several race conditions reported by valgrind

Might fix #374

CC -Fischer - [ ]  General Comment - I'll let the test run for a couple of hours and report whether it runs fine. - -Fischer I've run it locally for a few hours with no crashes, any crash on your side? Otherwise I'm merging this... - No crashes here for 3 or 4 hours .. Go ahead and merge I'd say :) - :+1: - -Fischer thanks for testing! - [x]  File: src/libYARP_serversql/src/NameServiceOnTriples.h:L57-69 - This might be a good chance to also use proper initializers for the other attributes .. - :+1: Done, thanks

  

该提问来源于开源项目:robotology/yarp

  • 点赞
  • 写回答
  • 关注问题
  • 收藏
  • 复制链接分享
  • 邀请回答

5条回答

  • weixin_39653442 weixin_39653442 4月前

    I'll let the test run for a couple of hours and report whether it runs fine.

    点赞 评论 复制链接分享
  • weixin_39668898 weixin_39668898 4月前

    -Fischer I've run it locally for a few hours with no crashes, any crash on your side? Otherwise I'm merging this...

    点赞 评论 复制链接分享
  • weixin_39653442 weixin_39653442 4月前

    No crashes here for 3 or 4 hours .. Go ahead and merge I'd say :) :+1:

    点赞 评论 复制链接分享
  • weixin_39653442 weixin_39653442 4月前

    :+1:

    点赞 评论 复制链接分享
  • weixin_39668898 weixin_39668898 4月前

    -Fischer thanks for testing!

    点赞 评论 复制链接分享

相关推荐