Skip to content

Add additional memory allocation check lalpulsar_spec_avg_long

Evan Goetz requested to merge evan-goetz/lalsuite:specavglong-mem-alloc into master

Description

This is a simple addition for a memory allocation check on a variable that can potentially be too big for the host it is running on. If too big of memory is attempted to be allocated, the pointer returned is NULL, which this line checks for.

API Changes and Justification

Backwards Compatible Changes

  • This change does not modify any class/function/struct/type definitions in a public C header file or any Python class/function definitions
  • This change adds new classes/functions/structs/types to a public C header file or Python module

Backwards Incompatible Changes

  • This change modifies an existing class/function/struct/type definition in a public C header file or Python module
  • This change removes an existing class/function/struct/type from a public C header file or Python module

If any of the Backwards Incompatible check boxes are ticked please provide a justification why this change is necessary and why it needs to be done in a backwards incompatible way.

Review Status

Perhaps @karl-wette or @david-keitel can have a look at this additional check.

Merge request reports