Merge pull request #690 from alexcohn/bugfix/#634_fdsan

tanersener · web-flow · commit 3327609fde68 · 2021-03-07T11:53:53.000Z
fixing #634
diff --git a/android/app/src/main/cpp/saf_wrapper.c b/android/app/src/main/cpp/saf_wrapper.c
@@ -91,7 +91,6 @@ static AVIOContext *create_fd_avio_context(const char *filename, int flags) {
 static void close_fd_avio_context(AVIOContext *ctx) {
     if (fd_seek(ctx->opaque, 0, AVSEEK_SIZE) >= 0) {
         int fd = (int)ctx->opaque;
-        close(fd);
         closeParcelFileDescriptor(fd);
     }
     ctx->opaque = NULL;

Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,6 @@ static AVIOContext create_fd_avio_context(const char filename, int flags) {`
`91`	`91`	`static void close_fd_avio_context(AVIOContext *ctx) {`
`92`	`92`	`if (fd_seek(ctx->opaque, 0, AVSEEK_SIZE) >= 0) {`
`93`	`93`	`int fd = (int)ctx->opaque;`
`94`		`- close(fd);`
`95`	`94`	`closeParcelFileDescriptor(fd);`
`96`	`95`	`}`
`97`	`96`	`ctx->opaque = NULL;`