[dev.icinga.com #409] More s/malloc(size) + init/calloc(1, size)/ replacements #197

Closed
icinga-migration opened this Issue Apr 30, 2010 · 2 comments

Projects

None yet

1 participant

@icinga-migration
Member

This issue has been migrated from Redmine: https://dev.icinga.com/issues/409

Created by mfriedrich on 2010-04-30 21:31:32 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2010-05-19 11:45:10 +00:00)
Target Version: 1.0.2
Last Update: 2010-05-19 11:45:10 +00:00 (in Redmine)


http://git.op5.org/git/?p=nagios.git;a=commit;h=e61679d14bb614167d835fd33dcba6c098b97539

More s/malloc(size) + init/calloc(1, size)/ replacements

This time in common/comments.c and common/downtime.c. Since we
really don't want to use explicit initialization on memory that
in all likelihood is already nul'ed out, as memory returned by
calloc() always is, and memory returned from malloc() that has
not been re-used by libc but comes straight from the kernel is,
this is a good thing.

Signed-off-by: Andreas Ericsson 

Changesets

2010-05-12 20:11:38 +00:00 by mfriedrich 2fcb5b5

more s/malloc(size) + init/calloc(1, size)/ replacements (Andreas Ericsson)

[--snip--]
This time in common/comments.c and common/downtime.c. Since we
really don't want to use explicit initialization on memory that
in all likelihood is already nul'ed out, as memory returned by
calloc() always is, and memory returned from malloc() that has
not been re-used by libc but comes straight from the kernel is,
this is a good thing.
[--snip--]

fixes #409
Member

Updated by mfriedrich on 2010-05-12 20:09:50 +00:00

  • Assigned to set to mfriedrich
  • Target Version set to 1.0.2
Member

Updated by mfriedrich on 2010-05-19 11:45:10 +00:00

  • Status changed from New to Resolved
  • Done % changed from 0 to 100

Applied in changeset commit:"2fcb5b5f7c684e60729999b2ca318e6d21eeec68".

@icinga-migration icinga-migration added this to the 1.0.2 milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment