Restructure the structs/unions involved in SRPC argument passing. This will...

tests/contest_issues/issue52.c

 /*
  * Copyright 2008 The Native Client Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can
  * be found in the LICENSE file.
  */
 
 /*
  * Simple test for simple rpc.
  */
 
@@ skipping 16 matching lines @@
 /*
  *  Return a clever string.
  */
 void HelloWorld(NaClSrpcRpc *rpc,
                 NaClSrpcArg **in_args,
                 NaClSrpcArg **out_args,
                 NaClSrpcClosure *done) {
   /*
    * Strdup must be used because the SRPC layer frees the string passed to it.
    */
-  out_args[0]->u.sval.str = strdup("hello, world.");
+  out_args[0]->arrays.str = strdup("hello, world.");
   rpc->result = NACL_SRPC_RESULT_OK;
   done->Run(done);
 }
 
 const struct NaClSrpcHandlerDesc srpc_methods[] = {
   { "fortytwo::i", FortyTwo },
   { "helloworld::sssssssssssssssssssssssssssssssssssssssss"
     "sssssssssssssssssssssssssssssssssssssssssssssssssssss"
     "sssssssssssssssssssssssssssssssssssssssssssssssssssss"
     "sssssssssssssssssssssssssssssssssssssssssssssssssssss",
     HelloWorld },
   { NULL, NULL },
 };
 
 int main() {
   if (!NaClSrpcModuleInit()) {
     return 1;
   }
   if (!NaClSrpcAcceptClientConnection(srpc_methods)) {
     return 1;
   }
   NaClSrpcModuleFini();
   return 0;
 }