Project

General

Profile

Actions
Copy link

Bug #102

closed

Fix signal handler

Added by Markus Elfring almost 14 years ago. Updated about 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Regression:
Severity:
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

This issue corresponds to my previous "feature request".

The functions "fprintf" and "exit" do not belong to the list of async-signal-safe functions.
See section "2.4.3 Signal Actions" from the document "2.4 Signal Concepts".

Should the corresponding calls be replaced by safe alternatives in your function "handler"?
source:src/mount/mtab.c@913315705b194edb2582911b9fcf9e97c6115bbf#L70

Actions
Copy link
 #1

Updated by Sage Weil almost 14 years ago

Sure, assuming there are in fact safe alternatives to fprintf.

If you do fix this, considering fixing nfs-utils' too mount.nfs too, as that is where this code came from.

Actions
Copy link
 #2

Updated by Sage Weil about 13 years ago

  • Status changed from New to Closed
Actions
Copy link

Also available in: Atom PDF