Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

found a bug in the file CTK/Plugins/org.commontk.eventadmin/tasks /ctkEASyncDeliverTasks.tpp #1156

Open
HBing110 opened this issue Nov 21, 2023 · 1 comment

Comments

@HBing110
Copy link

HBing110 commented Nov 21, 2023

CTK/Plugins/org.commontk.eventadmin/tasks /ctkEASyncDeliverTasks.tpp

139 line

After the entire project is initiated, there is only one instance of syncMasterThread. However, when multiple threads execute simultaneously, data may become disordered or duplicated.

@HBing110 HBing110 changed the title find a bug in CTK/Plugins/org.commontk.eventadmin/tasks /ctkEASyncDeliverTasks.tpp found a bug in the file CTK/Plugins/org.commontk.eventadmin/tasks /ctkEASyncDeliverTasks.tpp Nov 21, 2023
@HBing110 HBing110 closed this as not planned Won't fix, can't repro, duplicate, stale Nov 22, 2023
@HBing110
Copy link
Author

1、
I added a line before line 139: ctkEASyncMasterThread* syncMasterThread = new ctkEASyncMasterThread(); Running with high concurrency, the data is fine, but I haven't found a suitable place to release memory.

2、
Modify line 139:
{
QMutexLocker l(&mutex1);
syncMasterThread->syncRun(&runnable);
}
Inefficient, simultaneous nested message sending causing deadlock.

Hopes for assistance in resolving this bug. . Thank you.

@HBing110 HBing110 reopened this Nov 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant