Skip to content

Commit

Permalink
Update agent/lib_php_amqplib.c
Browse files Browse the repository at this point in the history
Co-authored-by: Michal Nowacki <[email protected]>
  • Loading branch information
zsistla and lavarou authored Feb 5, 2025
1 parent 127e62a commit a8569a2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion agent/lib_php_amqplib.c
Original file line number Diff line number Diff line change
Expand Up @@ -675,7 +675,7 @@ NR_PHP_WRAPPER(nr_rabbitmq_basic_get) {
*The retval should be an AMQPMessage. nr_php_is_zval_* ops do NULL checks
* as well.
*/
if (nr_php_is_zval_valid_object(*retval_ptr)) {
if (NULL != retval_ptr && nr_php_is_zval_valid_object(*retval_ptr)) {

Check failure

Code scanning / CodeQL

Redundant null check due to previous dereference High

This null check is redundant because
the value is dereferenced
in any case.
This null check is redundant because
the value is dereferenced
in any case.
This null check is redundant because
the value is dereferenced
in any case.
/*
* Get the exchange and routing key from the AMQPMessage
*/
Expand Down

0 comments on commit a8569a2

Please sign in to comment.