From 528e750f7be39e0f747fcfd0398763346d09097a Mon Sep 17 00:00:00 2001 From: Jack Nagel Date: Sat, 13 Jul 2013 16:38:30 -0500 Subject: [PATCH] Handle NULs in link targets in Pathname#resolved_path_exists? Fixes Homebrew/homebrew#19475. Fixes Homebrew/homebrew#21184. --- Library/Homebrew/extend/pathname.rb | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/Library/Homebrew/extend/pathname.rb b/Library/Homebrew/extend/pathname.rb index 6e582c7030..3d9fa4c949 100644 --- a/Library/Homebrew/extend/pathname.rb +++ b/Library/Homebrew/extend/pathname.rb @@ -253,7 +253,12 @@ class Pathname end def resolved_path_exists? - (dirname+readlink).exist? + link = readlink + rescue ArgumentError + # The link target contains NUL bytes + false + else + (dirname+link).exist? end # perhaps confusingly, this Pathname object becomes the symlink pointing to