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

Coverity - Calling "fclose" without checking return value #1743

Closed
danielwangksu opened this issue Jul 8, 2019 · 0 comments · Fixed by #1750
Closed

Coverity - Calling "fclose" without checking return value #1743

danielwangksu opened this issue Jul 8, 2019 · 0 comments · Fixed by #1750

Comments

@danielwangksu
Copy link
Contributor

Event check_return: | Calling "fclose" without checking return value

ros_comm/tools/rosbag_storage/src/chunked_file.cpp

Line 103 in 29053c4

fclose(file_);

artivis added a commit to artivis/ros_comm that referenced this issue Jul 10, 2019
@artivis
fix ros#1743
857cf2a
artivis added a commit to artivis/ros_comm that referenced this issue Jul 10, 2019
@artivis
check fclose returns 0
de05aed 
  fix ros#1743
@artivis artivis mentioned this issue Jul 10, 2019
Merged
mikepurvis pushed a commit that referenced this issue Jul 29, 2019
@artivis @mikepurvis
Check for fclose returning 0 in rosbag_storage (#1750)
d18958f 
fix #1743
dirk-thomas pushed a commit that referenced this issue Aug 4, 2020
@artivis @dirk-thomas
Check for fclose returning 0 in rosbag_storage (#1750)
08150d9 
fix #1743
dirk-thomas pushed a commit that referenced this issue Aug 4, 2020
@artivis @dirk-thomas
Check for fclose returning 0 in rosbag_storage (#1750)
934bb15 
fix #1743
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Check for fclose returns 0 artivis/ros_comm
1 participant
@danielwangksu