Skip to content

Commit

Permalink
Bad error handling in acquire subscription
Browse files Browse the repository at this point in the history
  • Loading branch information
bretambrose committed Jul 19, 2024
1 parent db285e3 commit 13faf2f
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion source/request-response/subscription_manager.c
Original file line number Diff line number Diff line change
Expand Up @@ -445,7 +445,12 @@ enum aws_acquire_subscription_result_type aws_rr_subscription_manager_acquire_su
const struct aws_rr_acquire_subscription_options *options) {

if (options->topic_filter_count == 0) {
return aws_raise_error(AWS_ERROR_INVALID_ARGUMENT);
AWS_LOGF_ERROR(
AWS_LS_MQTT_REQUEST_RESPONSE,
"request-response subscription manager - acquire_subscription for operation %" PRIu64
" with no topic filters",
options->operation_id);
return AASRT_FAILURE;
}

/*
Expand Down

0 comments on commit 13faf2f

Please sign in to comment.