[PATCH] D59005: Fix memory leak in CreateSigAltStack

Pietro Fezzardi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 6 13:29:51 PST 2019


fez marked an inline comment as done.
fez added a comment.

But now I'm puzzled by this check.

  // If the handlers are already registered, we're done.
  if (NumRegisteredSignals.load() != 0)
    return;

This should prevent the call to CreateSigAltStack to be reached the second time.

I need to investigate further.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59005/new/

https://reviews.llvm.org/D59005





More information about the llvm-commits mailing list