Add support for file locking

runtime/bin/file.cc

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "bin/file.h"
 
 #include "bin/builtin.h"
 #include "bin/dartutils.h"
 #include "bin/io_buffer.h"
 #include "bin/utils.h"
@@ skipping 373 matching lines @@
   if (file->Flush()) {
     Dart_SetReturnValue(args, Dart_True());
   } else {
     Dart_Handle err = DartUtils::NewDartOSError();
     if (Dart_IsError(err)) Dart_PropagateError(err);
     Dart_SetReturnValue(args, err);
   }
 }
 
 
+void FUNCTION_NAME(File_Lock)(Dart_NativeArguments args) {
+  File* file = GetFilePointer(Dart_GetNativeArgument(args, 0));
+  ASSERT(file != NULL);
+  int64_t lock;

[Lasse Reichstein Nielsen, 2015/01/08 11:59:22]

Is this the type on all platforms? There is no lock_t type to use instead?

[Søren Gjesse, 2015/01/09 13:06:19]

The int64_t type is used to get the Dart int argument into C++ code. Below it is
checked to be in the range [0, 2] and then cast to the C++ enum File::LockType
which is then converted to the concrete platform type.

+  // Lock type arguments are checked in Dart code to be legal.

[Lasse Reichstein Nielsen, 2015/01/08 11:59:21]

legal -> valid.
It's not a law.

Don't trust Dart code, always check again in C code.

[Søren Gjesse, 2015/01/09 13:06:19]

I an also checking here. It they are not integers the "Illegal argument" below
will be thrown.

+  if (DartUtils::GetInt64Value(Dart_GetNativeArgument(args, 1), &lock)) {
+    int64_t start;
+    int64_t end;
+    if (lock >= File::kLockMin &&
+        lock <= File::kLockMax &&
+        DartUtils::GetInt64Value(Dart_GetNativeArgument(args, 2), &start) &&
+        DartUtils::GetInt64Value(Dart_GetNativeArgument(args, 3), &end)) {
+      if (file->Lock(static_cast<File::LockType>(lock), start, end)) {
+        Dart_SetReturnValue(args, Dart_True());
+      } else {
+        Dart_Handle err = DartUtils::NewDartOSError();
+        if (Dart_IsError(err)) Dart_PropagateError(err);
+        Dart_SetReturnValue(args, err);
+      }
+      return;
+    }
+  }
+
+  OSError os_error(-1, "Invalid argument", OSError::kUnknown);
+  Dart_Handle err = DartUtils::NewDartOSError(&os_error);
+  if (Dart_IsError(err)) Dart_PropagateError(err);
+  Dart_SetReturnValue(args, err);
+}
+
+
 void FUNCTION_NAME(File_Create)(Dart_NativeArguments args) {
   const char* str =
       DartUtils::GetStringValue(Dart_GetNativeArgument(args, 0));
   bool result = File::Create(str);
   if (result) {
     Dart_SetReturnValue(args, Dart_NewBoolean(result));
   } else {
     Dart_Handle err = DartUtils::NewDartOSError();
     if (Dart_IsError(err)) Dart_PropagateError(err);
     Dart_SetReturnValue(args, err);
@@ skipping 785 matching lines @@
       result->SetAt(i, new CObjectInt64(CObject::NewInt64(data[i])));
     }
     CObjectArray* wrapper = new CObjectArray(CObject::NewArray(2));
     wrapper->SetAt(0, new CObjectInt32(CObject::NewInt32(CObject::kSuccess)));
     wrapper->SetAt(1, result);
     return wrapper;
   }
   return CObject::IllegalArgumentError();
 }
 
+
+CObject* File::LockRequest(const CObjectArray& request) {
+  if (request.Length() == 4 &&
+      request[0]->IsIntptr() &&
+      request[1]->IsInt32OrInt64() &&
+      request[2]->IsInt32OrInt64() &&
+      request[3]->IsInt32OrInt64()) {
+    File* file = CObjectToFilePointer(request[0]);
+    ASSERT(file != NULL);
+    if (!file->IsClosed()) {
+      int64_t lock = CObjectInt32OrInt64ToInt64(request[1]);
+      int64_t start = CObjectInt32OrInt64ToInt64(request[2]);
+      int64_t end = CObjectInt32OrInt64ToInt64(request[3]);
+      if (file->Lock(static_cast<File::LockType>(lock), start, end)) {
+        return CObject::True();
+      } else {
+        return CObject::NewOSError();
+      }
+    } else {
+      return CObject::FileClosedError();
+    }
+  }
+  return CObject::IllegalArgumentError();
+}
+
 }  // namespace bin
 }  // namespace dart