Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 362542 - Fix one signal handling issue & initialize the notification array when doing the allocation
Summary: Fix one signal handling issue & initialize the notification array when doing ...
Status: RESOLVED FIXED
Alias: None
Product: PTP
Classification: Tools
Component: SCI (show other bugs)
Version: unspecified   Edit
Hardware: All Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: rong li CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-01 03:23 EDT by rong li CLA
Modified: 2012-11-06 04:47 EST (History)
0 users

See Also:


Attachments
A patch to fix one signal handling issue & initialize the notification array when doing the allocation (6.27 KB, patch)
2011-11-01 03:29 EDT, rong li CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description rong li CLA 2011-11-01 03:23:57 EDT
Build Identifier: 

In this patch, it mainly fixed 3 kind of issues:

1. Fixed one signal handling issues: The old sigmask needs to be restored if the pthread_create failed.

2. The notification array "serialTest[serialNum]" needs to be set to the initial data when doing the allocate. Otherwise, when the data would be incorrect when the array is reused after multiple times.

3. Add codes to catch certain memory allocation exceptions

4. Add correct error check, and return the errcode back when the "feAgent->syncWait()" got an error or the connection handler failed.

Reproducible: Always
Comment 1 rong li CLA 2011-11-01 03:29:56 EDT
Created attachment 206249 [details]
A patch to fix one signal handling issue & initialize the notification array when doing the allocation
Comment 2 rong li CLA 2012-11-06 04:46:47 EST
Have applied the patch into master branch.