Per Stew's suggestions in code review:

* Added assert()s in command_reader on field and value sizes
* Changed from string to bytes scalar types in transaction.proto definitions for Command
* Ensure writes in network byte order for the length of the written protos
* Fixed unclear cur_field vs. current_field in TransactionServices methods
* Remove delete calls for mem_root allocated Strings