My bad, I didn't actually notice the first time that the version variable in 
get_version is declared as static, which means that the process was spawned 
just once, although this could cause threading issues, it should be fairly 
safe, the only problem was the extra "g_free(version)", which should actually 
cause a segfault pretty much everytime (I wonder why it doesn't).

I think that this late in the cycle we should just remove the extra g_free and 
perhaps apply this thread-safe fix after release.
-- 
https://code.launchpad.net/~jjardon/indicator-datetime/fix-844741/+merge/77735
Your team ayatana-commits is subscribed to branch lp:indicator-datetime.

_______________________________________________
Mailing list: https://launchpad.net/~ayatana-commits
Post to     : ayatana-commits@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ayatana-commits
More help   : https://help.launchpad.net/ListHelp

Reply via email to