Skip to content

NULL usage in compress.c does not check if it is defined and may not be on a given platform #1197

Description

@dalvescb

The usage of NULL in compress.c as introduced in commit 5c47755 seems error prone. There is no include for defining NULL on all platforms, on certain platforms (i.e., linux etc) the use of #include <stddef.h> in the check to define z_size_t in zconf.h has the side effect of defining NULL. However this is not the case on all platforms (see rust-lang/libz-sys#259 for example)

This should perhaps be replaced with Z_NULL?

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions