Fix dup and dup2 in nacl-mounts

Fix dup and dup2 in nacl-mounts

Actually wrap it and fix up implementations so they work properly.

TEST=DupTest.Simple
Committed: https://code.google.com/p/naclports/source/detail?r=615

[davidben, 2012-08-05 20:57:03]

Not really sure what the old code was doing, but this should work. :-)

[eugenis, 2012-08-09 11:10:29]

LGTM

https://chromiumcodereview.appspot.com/10832154/diff/5001/libraries/nacl-moun...
File libraries/nacl-mounts/base/KernelProxy.cc (right):

https://chromiumcodereview.appspot.com/10832154/diff/5001/libraries/nacl-moun...
libraries/nacl-mounts/base/KernelProxy.cc:479: return 0;
return newfd

https://chromiumcodereview.appspot.com/10832154/diff/5001/libraries/nacl-moun...
libraries/nacl-mounts/base/KernelProxy.cc:487: return 0;
return newfd;

[davidben, 2012-08-10 01:09:04]

https://chromiumcodereview.appspot.com/10832154/diff/5001/libraries/nacl-moun...
File libraries/nacl-mounts/base/KernelProxy.cc (right):

https://chromiumcodereview.appspot.com/10832154/diff/5001/libraries/nacl-moun...
libraries/nacl-mounts/base/KernelProxy.cc:479: return 0;
On 2012/08/09 11:10:29, eugenis wrote:
> return newfd

Done.

https://chromiumcodereview.appspot.com/10832154/diff/5001/libraries/nacl-moun...
libraries/nacl-mounts/base/KernelProxy.cc:487: return 0;
On 2012/08/09 11:10:29, eugenis wrote:
> return newfd;

Done. (The IRT hook doesn't actually care and returns zero or errno, but yeah
it's probably better to match one or the other and not a mix.)

[Evgeniy Stepanov, 2012-08-13 17:12:37]

lgtm