diff options
author | Martijn Coenen <maco@google.com> | 2018-12-05 09:42:25 +0100 |
---|---|---|
committer | Inna Palant <ipalant@google.com> | 2019-01-16 08:55:14 -0800 |
commit | 20ac1203a3201ac3e6d05a19325f5569033f3d08 (patch) | |
tree | 3f7799148b31b1e3245b743b35592f0c185892f4 | |
parent | cddcc0d9cf7cbba53e54d67455737a3900ae5d9f (diff) | |
download | core-android-9.0.0_r35.tar.gz |
Export maximum number of fds/ints in a native_handle.android-9.0.0_r35android-9.0.0_r34pie-qpr2-release
So we can deserialize it consisently and safely.
Bug: 120084106
Test: builds
Change-Id: I0eafff70d3a7e4d732fe600a0052efb90108208d
-rw-r--r-- | libcutils/include/cutils/native_handle.h | 3 | ||||
-rw-r--r-- | libcutils/native_handle.cpp | 6 |
2 files changed, 5 insertions, 4 deletions
diff --git a/libcutils/include/cutils/native_handle.h b/libcutils/include/cutils/native_handle.h index 10f5bc094..f6cae360f 100644 --- a/libcutils/include/cutils/native_handle.h +++ b/libcutils/include/cutils/native_handle.h @@ -23,6 +23,9 @@ extern "C" { #endif +#define NATIVE_HANDLE_MAX_FDS 1024 +#define NATIVE_HANDLE_MAX_INTS 1024 + /* Declare a char array for use with native_handle_init */ #define NATIVE_HANDLE_DECLARE_STORAGE(name, maxFds, maxInts) \ alignas(native_handle_t) char (name)[ \ diff --git a/libcutils/native_handle.cpp b/libcutils/native_handle.cpp index 66f7a3d59..b409e5b9d 100644 --- a/libcutils/native_handle.cpp +++ b/libcutils/native_handle.cpp @@ -22,9 +22,6 @@ #include <string.h> #include <unistd.h> -static const int kMaxNativeFds = 1024; -static const int kMaxNativeInts = 1024; - native_handle_t* native_handle_init(char* storage, int numFds, int numInts) { if ((uintptr_t) storage % alignof(native_handle_t)) { errno = EINVAL; @@ -39,7 +36,8 @@ native_handle_t* native_handle_init(char* storage, int numFds, int numInts) { } native_handle_t* native_handle_create(int numFds, int numInts) { - if (numFds < 0 || numInts < 0 || numFds > kMaxNativeFds || numInts > kMaxNativeInts) { + if (numFds < 0 || numInts < 0 || numFds > NATIVE_HANDLE_MAX_FDS || + numInts > NATIVE_HANDLE_MAX_INTS) { errno = EINVAL; return NULL; } |