Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- int win32_pthread_join(pthread_t *thread, void **value_ptr) {
- int result;
- HANDLE h = OpenThread(SYNCHRONIZE, FALSE, *thread);
- REDIS_NOTUSED(value_ptr);
- switch (WaitForSingleObject(h, INFINITE)) {
- case WAIT_OBJECT_0:
- result = 0;
- case WAIT_ABANDONED:
- result = EINVAL;
- default:
- result = GetLastError();
- }
- CloseHandle(h);
- return result;
- }
- This suspicious code was found in Redis project by PVS-Studio static code analyzer.
- Warning message is:
- V519 The 'result' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 253, 255. win32fixes.c 255
- PVS-Studio is a static analyzer for detecting bugs in the source code of applications written in C, C++, C++11, C++/CX. Site: http://www.viva64.com/en/pvs-studio/
Advertisement
Add Comment
Please, Sign In to add comment