Skip to content

Commit

Permalink
compiler: check slice to pointer-to-array conversion element type
Browse files Browse the repository at this point in the history
When checking a slice to pointer-to-array conversion, I forgot to
verify that the elements types are identical.

For golang/go#395

Change-Id: I533ac52c0b390af96fce78a8c468ae9d8ad79da9
Reviewed-on: https://go-review.googlesource.com/c/gofrontend/+/339329
Trust: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Cherry Mui <cherryyz@google.com>
  • Loading branch information
ianlancetaylor committed Aug 3, 2021
1 parent 0a4d612 commit 5436180
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
5 changes: 4 additions & 1 deletion go/expressions.cc
Original file line number Diff line number Diff line change
Expand Up @@ -3962,7 +3962,10 @@ Type_conversion_expression::do_lower(Gogo*, Named_object*,
if (type->points_to() != NULL
&& type->points_to()->array_type() != NULL
&& !type->points_to()->is_slice_type()
&& val->type()->is_slice_type())
&& val->type()->is_slice_type()
&& Type::are_identical(type->points_to()->array_type()->element_type(),
val->type()->array_type()->element_type(),
0, NULL))
{
Temporary_statement* val_temp = NULL;
if (!val->is_multi_eval_safe())
Expand Down
4 changes: 3 additions & 1 deletion go/types.cc
Original file line number Diff line number Diff line change
Expand Up @@ -846,7 +846,9 @@ Type::are_convertible(const Type* lhs, const Type* rhs, std::string* reason)
if (rhs->is_slice_type()
&& lhs->points_to() != NULL
&& lhs->points_to()->array_type() != NULL
&& !lhs->points_to()->is_slice_type())
&& !lhs->points_to()->is_slice_type()
&& Type::are_identical(lhs->points_to()->array_type()->element_type(),
rhs->array_type()->element_type(), 0, reason))
return true;

// An unsafe.Pointer type may be converted to any pointer type or to
Expand Down

0 comments on commit 5436180

Please sign in to comment.