Skip to content

Commit 1a70c50

Browse files
authored
Fix a bunch of places we forget to aws_raise_error() (#111)
1 parent b7a96fd commit 1a70c50

File tree

4 files changed

+2
-22
lines changed

4 files changed

+2
-22
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ on:
77

88

99
env:
10-
BUILDER_VERSION: v0.9.43
10+
BUILDER_VERSION: v0.9.55
1111
BUILDER_SOURCE: releases
1212
BUILDER_HOST: https://d19elf31gohf1l.cloudfront.net
1313
PACKAGE_NAME: aws-c-event-stream

source/event_stream_channel_handler.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -447,7 +447,7 @@ static int s_process_write_message(
447447
(void)slot;
448448
(void)message;
449449
AWS_FATAL_ASSERT(!"The event-stream-channel-handler is not designed to be a mid-channel handler.");
450-
return AWS_OP_ERR;
450+
return aws_raise_error(AWS_ERROR_UNIMPLEMENTED);
451451
}
452452

453453
static int s_increment_read_window(struct aws_channel_handler *handler, struct aws_channel_slot *slot, size_t size) {

source/event_stream_rpc_client.c

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -499,16 +499,6 @@ static int s_send_protocol_message(
499499

500500
struct event_stream_connection_send_message_args *args =
501501
aws_mem_calloc(connection->allocator, 1, sizeof(struct event_stream_connection_send_message_args));
502-
503-
if (!message_args) {
504-
AWS_LOGF_ERROR(
505-
AWS_LS_EVENT_STREAM_RPC_CLIENT,
506-
"id=%p: failed to allocate callback arguments %s.",
507-
(void *)connection,
508-
aws_error_debug_str(aws_last_error()));
509-
return AWS_OP_ERR;
510-
}
511-
512502
args->allocator = connection->allocator;
513503
args->user_data = user_data;
514504
args->message_type = message_args->message_type;

source/event_stream_rpc_server.c

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -570,16 +570,6 @@ static int s_send_protocol_message(
570570

571571
struct event_stream_connection_send_message_args *args =
572572
aws_mem_calloc(connection->allocator, 1, sizeof(struct event_stream_connection_send_message_args));
573-
574-
if (!message_args) {
575-
AWS_LOGF_ERROR(
576-
AWS_LS_EVENT_STREAM_RPC_SERVER,
577-
"id=%p: allocation of callback args failed with error %s",
578-
(void *)connection,
579-
aws_error_debug_str(aws_last_error()));
580-
return AWS_OP_ERR;
581-
}
582-
583573
args->allocator = connection->allocator;
584574
args->user_data = user_data;
585575
args->message_type = message_args->message_type;

0 commit comments

Comments
 (0)