Skip to content

Commit bcf05ea

Browse files
committed
src: replace emit warning with DCHECK
Replace emitting of a process warning by a DCHECK because the use after free in http parser is fixed now. Refs: nodejs#61995 (comment) Refs: nodejs#62095
1 parent 5f02bdb commit bcf05ea

1 file changed

Lines changed: 2 additions & 10 deletions

File tree

src/async_wrap.cc

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,6 @@
2525
#include "env-inl.h"
2626
#include "node_errors.h"
2727
#include "node_external_reference.h"
28-
#ifdef DEBUG
29-
#include <node_process-inl.h>
30-
#endif
3128
#include "tracing/traced_value.h"
3229
#include "util-inl.h"
3330

@@ -677,13 +674,8 @@ MaybeLocal<Value> AsyncWrap::MakeCallback(const Local<Function> cb,
677674
Local<Value>* argv) {
678675
EmitTraceEventBefore();
679676

680-
#ifdef DEBUG
681-
if (context_frame().IsEmpty()) {
682-
ProcessEmitWarning(env(),
683-
"MakeCallback() called without context_frame, "
684-
"likely use after destroy of AsyncWrap.");
685-
}
686-
#endif
677+
// If this check fails it indicates an use after-free.
678+
DCHECK(!context_frame().IsEmpty());
687679

688680
ProviderType provider = provider_type();
689681
async_context context { get_async_id(), get_trigger_async_id() };

0 commit comments

Comments
 (0)