Cpp bindings: Return false from Validator::Accept() on unrecognized message or invalid flags

mojo/public/tools/bindings/generators/cpp_templates/interface_definition.tmpl

 {%- import "interface_macros.tmpl" as interface_macros %}
 {%- import "struct_macros.tmpl" as struct_macros %}
 {%- set class_name = interface.name %}
 {%- set proxy_name = interface.name ~ "Proxy" %}
 {%- set namespace_as_string = "%s"|format(namespace|replace(".","::")) %}
 
 {%- macro alloc_params(struct) %}
 {%-   for param in struct.packed.packed_fields_in_ordinal_order %}
   {{param.field.kind|cpp_result_type}} p_{{param.field.name}}{};
 {%-   endfor %}
@@ skipping 255 matching lines @@
       mojo::internal::BoundsChecker bounds_checker(
           message->payload(), message->payload_num_bytes(),
           message->handles()->size());
       if (!internal::{{class_name}}_{{method.name}}_ResponseParams_Data::Validate(
               message->payload(), &bounds_checker)) {
         return false;
       }
       break;
     }
 {%-   endfor %}
+    default: {
+      // Unrecognized message.
+      return false;

[yzshen1, 2015/02/17 19:39:35]

You also need to do the same for RequestValidator.

Besides, we now can reject messages if the flags are unexpected, too.

[rudominer, 2015/02/23 19:12:22]

Done
Haven't done this part yet.

+    }
   }
 {%- endif %}
 
   assert(sink_);
   return sink_->Accept(message);
 }
 {%- endif -%}