Ticket #5734 enhancement closed fixed

Opened 22 months ago

Last modified 21 months ago

attempt to quash a warning by storing the result of write() in _sigchld.c is unsuccessful

Reported by: glyph Owned by: glyph
Priority: normal Milestone:
Component: core Keywords:
Cc: Branch: branches/store-write-result-sigchld-5734
(diff, github, buildbot, log)
Author: ashfall Launchpad Bug:


We should apply this:

  • twisted/internet/_sigchld.c

    1313static void got_signal(int sig) { 
    1414    int saved_errno = errno; 
    15     int ignored_result; 
     15    ssize_t ignored_result; 
    1717    /* write() errors are unhandled.  If the buffer is full, we don't 
    1818     * care.  What about other errors? */ 

since, in fact, write() returns ssize_t and not int. (Sometimes they're the same thing, sometimes not.)

Change History


Changed 22 months ago by glyph

  • keywords review added


Changed 22 months ago by ashfall

  • owner set to ashfall
  • status changed from new to assigned


Changed 22 months ago by ashfall

  • branch set to branches/store-write-result-sigchld-5734
  • branch_author set to ashfall

(In [34658]) Branching to 'store-write-result-sigchld-5734'


Changed 22 months ago by ashfall

  • status changed from assigned to new
  • owner changed from ashfall to glyph
  • keywords review removed

Looks good to me. Please add a news file and merge.

 Build Results


Changed 21 months ago by glyph

  • status changed from new to closed
  • resolution set to fixed

(In [34829]) Merge store-write-result-sigchld-5734

Author: glyph

Reviewer: ashfall

Fixes: #5734

Fix a minor defect in _sigchld.c; store the result of a function in a container declared to be of the correct size, rather than the incorrect size.

Note: See TracTickets for help on using tickets.