TimeZoneInfo: Determine readlink result whether should be validated or not

[jozkee]

From @pedrobsaila #72258 (comment):

libraries\System.Private.CoreLib\src\System\TimeZoneInfo.Unix.NonAndroid.cs : seems to handle case where link is not resolved. I'm not sure whether I should validate or not.

runtime/src/libraries/System.Private.CoreLib/src/System/TimeZoneInfo.Unix.NonAndroid.cs

Lines 359 to 379 in a5f3676

	if (File.Exists(tzFilePath))
	{
	try
	{
	rawData = File.ReadAllBytes(tzFilePath);
	if (string.IsNullOrEmpty(id))
	{
	id = FindTimeZoneIdUsingReadLink(tzFilePath);
	if (string.IsNullOrEmpty(id))
	{
	id = FindTimeZoneId(rawData);
	}
	}
	return true;
	}
	catch (IOException) { }
	catch (SecurityException) { }
	catch (UnauthorizedAccessException) { }
	}
	return false;

I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

Tagging subscribers to this area: @dotnet/area-system-runtime
See info in area-owners.md if you want to be subscribed.

Issue Details

---

From @pedrobsaila #72258 (comment):

libraries\System.Private.CoreLib\src\System\TimeZoneInfo.Unix.NonAndroid.cs : seems to handle case where link is not resolved. I'm not sure whether I should validate or not.

runtime/src/libraries/System.Private.CoreLib/src/System/TimeZoneInfo.Unix.NonAndroid.cs

Lines 359 to 379 in a5f3676

	if (File.Exists(tzFilePath))
	{
	try
	{
	rawData = File.ReadAllBytes(tzFilePath);
	if (string.IsNullOrEmpty(id))
	{
	id = FindTimeZoneIdUsingReadLink(tzFilePath);
	if (string.IsNullOrEmpty(id))
	{
	id = FindTimeZoneId(rawData);
	}
	}
	return true;
	}
	catch (IOException) { }
	catch (SecurityException) { }
	catch (UnauthorizedAccessException) { }
	}
	return false;

Author:	Jozkee
Assignees:	-
Labels:	area-System.Runtime, untriaged
Milestone:	-

[jeffhandley]

@jozkee I'm moving this to Future, but if it's fixed during RC2 we could consider porting the fix into that branch.