balsoft/ligo
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Michelson: more robust checking of type alias annotations

Browse Source
This commit is contained in:
Alain Mebsout 2018-05-25 17:24:13 +02:00 committed by Benjamin Canou
parent 96e317f9d3
commit 382e06cf32
3 changed files with 52 additions and 38 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/bin_client/test/contracts/unpair_macro.tz
View File

@ -1,8 +1,9 @@
parameter unit;
storage unit;
code { UNIT @4; UNIT @3; UNIT @2; UNIT @1;
parameter (unit :param_unit);
storage (unit :u1);
code { DROP ;
UNIT :u4 @4; UNIT :u3 @3; UNIT :u2 @2; UNIT :u1 @1;
PAIR; UNPAIR @x1 @x2;
PPAIPAIR @p1 %x1 %x2 %x3 %x4; UNPPAIPAIR @uno @due @tre @quattro;
PAPAPAIR @p2 %x1 %x2 %x3 %x4; UNPAPAPAIR @un @deux @trois @quatre;
PAPPAIIR @p3 %x1 %x2 %x3 %x4; UNPAPPAIIR @one @two @three @four;
DROP; DROP; DROP; DROP; CAR; NIL operation; PAIR }
DIP { DROP; DROP; DROP }; NIL operation; PAIR }

3
src/proto_alpha/lib_protocol/src/script_ir_annot.mli
View File

@ -67,7 +67,8 @@ val gen_binding_access_annot :
binding_annot option -> binding_annot option
(** Merge type annotations.
@returns {!Inconsistent_type_annotations} if they are both present and different *)
@returns an error {!Inconsistent_type_annotations} if they are both present
and different *)
val merge_type_annot :
type_annot option -> type_annot option -> type_annot option tzresult

72
src/proto_alpha/lib_protocol/src/script_ir_translator.ml
View File

@ -1403,8 +1403,9 @@ and parse_returning
| (Typed ({ loc ; aft = (Item_t (ty, Empty_t, _) as stack_ty) ; _ } as descr), gas) ->
trace
(Bad_return (loc, stack_ty, ret))
(Lwt.return (ty_eq ty ret)) >>=? fun Eq ->
return ((Lam (descr, strip_locations script_instr) : (arg, ret) lambda), gas)
(Lwt.return (ty_eq ty ret) >>=? fun Eq ->
Lwt.return (merge_types loc ty ret) >>=? fun _ret ->
return ((Lam (descr, strip_locations script_instr) : (arg, ret) lambda), gas))
| (Typed { loc ; aft = stack_ty ; _ }, _gas) ->
fail (Bad_return (loc, stack_ty, ret))
| (Failed { descr }, gas) ->
@ -1596,11 +1597,12 @@ and parse_instr
body (Item_t (elt, starting_rest, elt_annot)) >>=? begin fun (judgement, ctxt) ->
match judgement with
| Typed ({ aft = Item_t (ret, rest, _) ; _ } as ibody) ->
trace
(Invalid_map_body (loc, ibody.aft))
(Lwt.return (stack_ty_eq 1 rest starting_rest)) >>=? fun Eq ->
let invalid_map_body = Invalid_map_body (loc, ibody.aft) in
trace invalid_map_body
(Lwt.return (stack_ty_eq 1 rest starting_rest) >>=? fun Eq ->
Lwt.return (merge_stacks loc rest starting_rest) >>=? fun rest ->
typed ctxt loc (List_map ibody)
(Item_t (List_t (ret, list_ty_name), rest, ret_annot))
(Item_t (List_t (ret, list_ty_name), rest, ret_annot)))
| Typed { aft ; _ } -> fail (Invalid_map_body (loc, aft))
| Failed _ -> fail (Invalid_map_block_fail loc)
end
@ -1615,10 +1617,11 @@ and parse_instr
body (Item_t (elt, rest, elt_annot)) >>=? begin fun (judgement, ctxt) ->
match judgement with
| Typed ({ aft ; _ } as ibody) ->
trace
(Invalid_iter_body (loc, rest, ibody.aft))
(Lwt.return (stack_ty_eq 1 aft rest)) >>=? fun Eq ->
typed ctxt loc (List_iter ibody) rest
let invalid_iter_body = Invalid_iter_body (loc, rest, ibody.aft) in
trace invalid_iter_body
(Lwt.return (stack_ty_eq 1 aft rest) >>=? fun Eq ->
Lwt.return (merge_stacks loc aft rest) >>=? fun rest ->
typed ctxt loc (List_iter ibody) rest)
| Failed { descr } ->
typed ctxt loc (List_iter (descr rest)) rest
end
@ -1640,10 +1643,11 @@ and parse_instr
body (Item_t (elt, rest, elt_annot)) >>=? begin fun (judgement, ctxt) ->
match judgement with
| Typed ({ aft ; _ } as ibody) ->
trace
(Invalid_iter_body (loc, rest, ibody.aft))
(Lwt.return (stack_ty_eq 1 aft rest)) >>=? fun Eq ->
typed ctxt loc (Set_iter ibody) rest
let invalid_iter_body = Invalid_iter_body (loc, rest, ibody.aft) in
trace invalid_iter_body
(Lwt.return (stack_ty_eq 1 aft rest) >>=? fun Eq ->
Lwt.return (merge_stacks loc aft rest) >>=? fun rest ->
typed ctxt loc (Set_iter ibody) rest)
| Failed { descr } ->
typed ctxt loc (Set_iter (descr rest)) rest
end
@ -1681,11 +1685,12 @@ and parse_instr
body (Item_t (Pair_t ((k, key_field), (elt, elt_field), None), starting_rest, None)) >>=? begin fun (judgement, ctxt) ->
match judgement with
| Typed ({ aft = Item_t (ret, rest, _) ; _ } as ibody) ->
trace
(Invalid_map_body (loc, ibody.aft))
(Lwt.return (stack_ty_eq 1 rest starting_rest)) >>=? fun Eq ->
let invalid_map_body = Invalid_map_body (loc, ibody.aft) in
trace invalid_map_body
(Lwt.return (stack_ty_eq 1 rest starting_rest) >>=? fun Eq ->
Lwt.return (merge_stacks loc rest starting_rest) >>=? fun rest ->
typed ctxt loc (Map_map ibody)
(Item_t (Map_t (ck, ret, ty_name), rest, ret_annot))
(Item_t (Map_t (ck, ret, ty_name), rest, ret_annot)))
| Typed { aft ; _ } -> fail (Invalid_map_body (loc, aft))
| Failed _ -> fail (Invalid_map_block_fail loc)
end
@ -1700,10 +1705,11 @@ and parse_instr
(Item_t (Pair_t ((key, key_field), (element_ty, elt_field), None), rest, None))
>>=? begin fun (judgement, ctxt) -> match judgement with
| Typed ({ aft ; _ } as ibody) ->
trace
(Invalid_iter_body (loc, rest, ibody.aft))
(Lwt.return (stack_ty_eq 1 aft rest)) >>=? fun Eq ->
typed ctxt loc (Map_iter ibody) rest
let invalid_iter_body = Invalid_iter_body (loc, rest, ibody.aft) in
trace invalid_iter_body
(Lwt.return (stack_ty_eq 1 aft rest) >>=? fun Eq ->
Lwt.return (merge_stacks loc aft rest) >>=? fun rest ->
typed ctxt loc (Map_iter ibody) rest)
| Failed { descr } ->
typed ctxt loc (Map_iter (descr rest)) rest
end
@ -1810,10 +1816,11 @@ and parse_instr
rest >>=? begin fun (judgement, ctxt) ->
match judgement with
| Typed ibody ->
trace
(Unmatched_branches (loc, ibody.aft, stack))
(Lwt.return (stack_ty_eq 1 ibody.aft stack)) >>=? fun Eq ->
typed ctxt loc (Loop ibody) rest
let unmatched_branches = Unmatched_branches (loc, ibody.aft, stack) in
trace unmatched_branches
(Lwt.return (stack_ty_eq 1 ibody.aft stack) >>=? fun Eq ->
Lwt.return (merge_stacks loc ibody.aft stack) >>=? fun _stack ->
typed ctxt loc (Loop ibody) rest)
| Failed { descr } ->
let ibody = descr stack in
typed ctxt loc (Loop ibody) rest
@ -1828,10 +1835,11 @@ and parse_instr
parse_instr ?type_logger tc_context ctxt body
(Item_t (tl, rest, l_annot)) >>=? begin fun (judgement, ctxt) -> match judgement with
| Typed ibody ->
trace
(Unmatched_branches (loc, ibody.aft, stack))
(Lwt.return (stack_ty_eq 1 ibody.aft stack)) >>=? fun Eq ->
typed ctxt loc (Loop_left ibody) (Item_t (tr, rest, r_annot))
let unmatched_branches = Unmatched_branches (loc, ibody.aft, stack) in
trace unmatched_branches
(Lwt.return (stack_ty_eq 1 ibody.aft stack) >>=? fun Eq ->
Lwt.return (merge_stacks loc ibody.aft stack) >>=? fun _stack ->
typed ctxt loc (Loop_left ibody) (Item_t (tr, rest, r_annot)))
| Failed { descr } ->
let ibody = descr stack in
typed ctxt loc (Loop_left ibody) (Item_t (tr, rest, r_annot))
@ -2292,8 +2300,11 @@ and parse_instr
fun (Lam ({ bef = Item_t (arg, Empty_t, _) ;
aft = Item_t (ret, Empty_t, _) ; _ }, _) as lambda, ctxt) ->
Lwt.return @@ ty_eq arg arg_type_full >>=? fun Eq ->
Lwt.return @@ merge_types loc arg arg_type_full >>=? fun _ ->
Lwt.return @@ ty_eq ret ret_type_full >>=? fun Eq ->
Lwt.return @@ merge_types loc ret ret_type_full >>=? fun _ ->
Lwt.return @@ ty_eq storage_type ginit >>=? fun Eq ->
Lwt.return @@ merge_types loc storage_type ginit >>=? fun _ ->
typed ctxt loc (Create_contract (storage_type, arg_type, lambda))
(Item_t (Operation_t None, Item_t (Address_t None, rest, addr_annot), op_annot))
| Prim (loc, I_NOW, [], annot),
@ -2458,6 +2469,7 @@ and parse_contract
parse_toplevel code >>? fun (arg_type, _, _) ->
parse_ty ~allow_big_map:false ~allow_operation:false arg_type >>? fun (Ex_ty targ) ->
ty_eq targ arg >>? fun Eq ->
merge_types loc targ arg >>? fun arg ->
let contract : arg typed_contract = (arg, contract) in
ok (ctxt, contract))