dotnet/wpf

Avoid calling Path.GetTempFileName

Open

#259 geöffnet am 11. Jan. 2019

Auf GitHub ansehen
 (3 Kommentare) (3 Reaktionen) (0 zugewiesene Personen)C# (6.683 Stars) (1.126 Forks)batch import
:construction: work in progressEnhancement RequestedGood First Issue

Beschreibung

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

Contributor Guide

Tech Stack
csharp
Domain
desktop
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Basic C#File I/O concepts
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
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.