Add bound to VBlob returned from VImage.getBlob (#215)

Author: lopcode

README.md

@@ -23,7 +23,7 @@ repositories {
 }
 
 dependencies {
-    implementation("app.photofox.vips-ffm:vips-ffm-core:1.9.6")
+    implementation("app.photofox.vips-ffm:vips-ffm-core:1.9.7")
 }
 ```
 

build.gradle.kts

@@ -1,3 +1,3 @@
 plugins {
-    kotlin("jvm") version "2.3.0" apply false
+    kotlin("jvm") version "2.3.21" apply false
 }

core/src/main/java/app/photofox/vipsffm/VImage.java

@@ -10454,7 +10454,14 @@ public VBlob getBlob(String name) {
     if (!VipsValidation.isValidPointer(outPointer)) {
       throw new VipsError("failed to read value of type blob from field: " + name);
     }
-    var blobAddress = outPointer.get(VipsRaw.C_POINTER, 0);
+    if (!VipsValidation.isValidPointer(outLengthPointer)) {
+      throw new VipsError("failed to read length pointer of type blob from field: " + name);
+    }
+    var blobLength = outLengthPointer.get(VipsRaw.C_LONG, 0);
+    if (blobLength <= 0) {
+      throw new VipsError("failed to read length of type blob from field: " + name);
+    }
+    var blobAddress = outPointer.get(VipsRaw.C_POINTER, 0).reinterpret(blobLength);
     return new VBlob(arena, blobAddress);
   }
 

generator/src/main/kotlin/vipsffm/GenerateVClasses.kt

@@ -1072,7 +1072,22 @@ object GenerateVClasses {
 
                         vblobType -> {
                             // void **
-                            this.addStatement("var blobAddress = outPointer.get(\$T.C_POINTER, 0)", vipsRawType)
+                            this.addCode(
+                                CodeBlock.builder()
+                                    .beginControlFlow("if (!\$T.isValidPointer(outLengthPointer))", vipsValidatorType)
+                                    .addStatement("throw new VipsError(\"failed to read length pointer of type $typeName from field: \" + name)")
+                                    .endControlFlow()
+                                    .build()
+                            )
+                            this.addStatement("var blobLength = outLengthPointer.get(\$T.C_LONG, 0)", vipsRawType)
+                            this.addCode(
+                                CodeBlock.builder()
+                                    .beginControlFlow("if (blobLength <= 0)")
+                                    .addStatement("throw new VipsError(\"failed to read length of type $typeName from field: \" + name)")
+                                    .endControlFlow()
+                                    .build()
+                            )
+                            this.addStatement("var blobAddress = outPointer.get(\$T.C_POINTER, 0).reinterpret(blobLength)", vipsRawType)
                             this.addStatement("return new VBlob(arena, blobAddress)")
                         }