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.

Sign up for GitHub

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

Jump to bottom

Error handling #5

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Error handling #5

wants to merge 5 commits into from

Conversation

mogeb
Copy link

@mogeb mogeb commented Mar 28, 2017

A shared library code shouldn't exit()

@mogeb
Add better error handling
4ce5fb0 
A shared object should never exit().

Signed-off-by: Mohamad Gebai <mohamad.gebai@gmail.com>
@mogeb
Free previous allocations if initialization fails
948d133 
Signed-off-by: Mohamad Gebai <mohamad.gebai@gmail.com>
@mogeb
Update example to use initialize return value
52cdf3d 
Signed-off-by: Mohamad Gebai <mohamad.gebai@gmail.com>
@mogeb
Use fclose instead of close for fdopen()
34fc9af 
Signed-off-by: Mohamad Gebai <mohamad.gebai@gmail.com>
@mogeb
Update test function to use return value of initialize()
0542269 
Signed-off-by: Mohamad Gebai <mohamad.gebai@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@mogeb