dotnet/wpf

Avoid calling Path.GetTempFileName

Open

#259 ouverte le 11 janv. 2019

Voir sur GitHub
 (3 commentaires) (3 réactions) (0 assignés)C# (6 683 stars) (1 126 forks)batch import
:construction: work in progressEnhancement RequestedGood First Issue

Description

System.Windows.Input.Cursor.LoadFromStream may throw IOException. DevDiv\FX\Rel\Net471Rel1Last\wpf\src\Core\CSharp\System\Windows\Input\Cursor.cs calls Path.GetTempFileName which will throw if the %TEMP% directory contains too many temp files (65k).

Reported by customer as VS Feedback issue

Guide contributeur

Stack technique
csharp
Domaine
desktop
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Basic C#File I/O concepts
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Investigate the file `DevDiv/FX/Rel/Net471Rel1Last/wpf/src/Core/CSharp/System/Windows/Input/Cursor.cs` to understand the current usage of Path.GetTempFileName. Check the 3 comments for any proposed solutions or maintainer guidance. Replace Path.GetTempFileName with a non throwing alternative like `Path.Combine(Path.GetTempPath(), Path.GetRandomFileName())` and handle potential exceptions gracefully.