Passes invalid paths to chardetect.py and has no fallback if it fails

Bug #1101448 reported by Sergey "Shnatsel" Davidoff
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Scratch
Fix Released
High
Sergey "Shnatsel" Davidoff

Bug Description

Scratch passes invalid paths to chardetect.py, which causes character detection to fail, e.g. on path "/home/serioga/Склад/Alex the Lion. Собрание сочинений.txt" (see console output attached)

Scratch has no fallback for such case, and displays blank file when character encoding detection fails (i.e. fails silently) and makes the user believe that the file is blank, which is not true (and this is probably a separate bug).

ProblemType: Bug
DistroRelease: elementary OS 0.2
Package: scratch-text-editor 1.1.1+r975-0+pkg39~precise1 [origin: LP-PPA-elementary-os-daily]
ProcVersionSignature: Ubuntu 3.5.0-19.30~precise1-generic 3.5.7
Uname: Linux 3.5.0-19-generic x86_64
ApportVersion: 2.0.1-0ubuntu15+elementary3~precise1
Architecture: amd64
CrashDB: scratch_text_editor
Date: Sat Jan 19 00:32:43 2013
ExecutablePath: /usr/bin/scratch-text-editor
GsettingsChanges:

InstallationMedia: elementary OS 0.2 "Luna" - Build amd64 LIVE Binary 20121001-15:16
MarkForUpload: True
ProcEnviron:
 PATH=(custom, no user)
 LANG=ru_RU.UTF-8
 SHELL=/bin/bash
SourcePackage: scratch-text-editor
UpgradeStatus: No upgrade log present (probably fresh install)

Related branches

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :
Changed in scratch:
status: New → In Progress
assignee: nobody → Sergey "Shnatsel" Davidoff (shnatsel)
summary: - Passes invalid paths to python-chardetect and has no fallback
+ Passes invalid paths to chardetect.py and has no fallback if it fails
description: updated
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Setting importance to hight because right now Scratch cannot display any files which have a space in the file name.

Changed in scratch:
importance: Undecided → High
Changed in scratch:
status: In Progress → Fix Committed
Cody Garver (codygarver)
Changed in scratch:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.