Opened 17 years ago

Closed 16 years ago

Last modified 15 years ago

#242 closed Defect (fixed)

Fix signal handler

Reported by: elfring Owned by: davea
Priority: Undetermined Milestone: 6.4
Component: BOINC - API Version:
Keywords: Cc:

Description

The following functions 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".)

  • fprintf()
  • getrusage()
  • longjmp()
  • perror()
  • pthread_equal()
  • pthread_mutex_trylock()
  • pthread_mutex_unlock()

Should the corresponding calls be replaced by safe alternatives in your functions "worker_signal_handler" and "restart_sig"?

How do you think about to specify the data type "sig_atomic_t" for the variables "options.direct_process_action", "boinc_status.suspended" and "in_critical_section"?

Change History (2)

comment:1 Changed 16 years ago by romw

Owner: changed from Bruce Allen to davea

comment:2 Changed 16 years ago by davea

Resolution: fixed
Status: newclosed

None of those functions are used in signal handlers anymore

Note: See TracTickets for help on using tickets.