GeezerVison is showing github.com — src: add nullptr check for session in DEBUG macro · nodejs/node@5dda33f
File tree
Expand file treeCollapse file tree
Open diff view settings
Expand file treeCollapse file tree
Open diff view settings
Collapse file
Copy file name to clipboardExpand all lines: src/node_http2.h +10-6Lines changed: 10 additions & 6 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -39,16 +39,20 @@ void inline debug_vfprintf(const char* format, ...) { |
39 | 39 | #define DEBUG_HTTP2(...) debug_vfprintf(__VA_ARGS__); |
40 | 40 | #define DEBUG_HTTP2SESSION(session, message) \ |
41 | 41 | do { \ |
42 | | - DEBUG_HTTP2("Http2Session %s (%.0lf) " message "\n", \ |
43 | | - session->TypeName(), \ |
44 | | - session->get_async_id()); \ |
| 42 | + if (session != nullptr) { \ |
| 43 | + DEBUG_HTTP2("Http2Session %s (%.0lf) " message "\n", \ |
| 44 | + session->TypeName(), \ |
| 45 | + session->get_async_id()); \ |
| 46 | + } \ |
45 | 47 | } while (0) |
46 | 48 | #define DEBUG_HTTP2SESSION2(session, message, ...) \ |
47 | 49 | do { \ |
48 | | - DEBUG_HTTP2("Http2Session %s (%.0lf) " message "\n", \ |
49 | | - session->TypeName(), \ |
50 | | - session->get_async_id(), \ |
| 50 | + if (session != nullptr) { \ |
| 51 | + DEBUG_HTTP2("Http2Session %s (%.0lf) " message "\n", \ |
| 52 | + session->TypeName(), \ |
| 53 | + session->get_async_id(), \ |
51 | 54 | __VA_ARGS__); \ |
| 55 | + } \ |
52 | 56 | } while (0) |
53 | 57 | #define DEBUG_HTTP2STREAM(stream, message) \ |
54 | 58 | do { \ |
|