Summary
The pullArtifact methods in Registry and OCILayout use the org.opencontainers.image.title annotation from a pulled manifest as a filename, resolving it against the caller supplied output directory without normalization or a containment check. A manifest publisher can set this annotation to a path that escapes the output directory, causing the SDK to write the layer's blob anywhere the JVM process can write.
Details
Two call sites are affected.
src/main/java/land/oras/Registry.java, pullLayer (reached from Registry.pullArtifact):
Path targetPath = path.resolve(layer.getAnnotations().get(Const.ANNOTATION_TITLE));
...
Files.copy(is, targetPath, StandardCopyOption.REPLACE_EXISTING);
src/main/java/land/oras/OCILayout.java, OCILayout.pullArtifact:
Files.copy(blobPath, path.resolve(layer.getAnnotations().get(Const.ANNOTATION_TITLE)));
The annotation comes from the remote manifest. Path.resolve treats an absolute argument as a full override of the base, and follows .. segments upward, so the annotation controls the destination. REPLACE_EXISTING overwrites files that exist at that destination.
The unpack branch of pullLayer (taken when the layer carries io.deis.oras.content.unpack=true) is not affected, because it dispatches through ArchiveUtils.untar / unzip, which apply outputPath.startsWith(normalizedTarget) after normalization. The non unpack branch and OCILayout.pullArtifact lack the equivalent check.
fetchBlob(ContainerRef, Path) is not affected. The caller passes the destination path and the title annotation is not consulted.
References
Summary
The
pullArtifactmethods inRegistryandOCILayoutuse theorg.opencontainers.image.titleannotation from a pulled manifest as a filename, resolving it against the caller supplied output directory without normalization or a containment check. A manifest publisher can set this annotation to a path that escapes the output directory, causing the SDK to write the layer's blob anywhere the JVM process can write.Details
Two call sites are affected.
src/main/java/land/oras/Registry.java,pullLayer(reached fromRegistry.pullArtifact):src/main/java/land/oras/OCILayout.java,OCILayout.pullArtifact:The annotation comes from the remote manifest.
Path.resolvetreats an absolute argument as a full override of the base, and follows..segments upward, so the annotation controls the destination.REPLACE_EXISTINGoverwrites files that exist at that destination.The unpack branch of
pullLayer(taken when the layer carriesio.deis.oras.content.unpack=true) is not affected, because it dispatches throughArchiveUtils.untar/unzip, which applyoutputPath.startsWith(normalizedTarget)after normalization. The non unpack branch andOCILayout.pullArtifactlack the equivalent check.fetchBlob(ContainerRef, Path)is not affected. The caller passes the destination path and the title annotation is not consulted.References