Trailing slash in .config/safe-rm don't work as expected

Bug #1398437 reported by lalit
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Safe-rm
Fix Released
Medium
François Marier

Bug Description

trailing slash in .config/safe-rm crashes safe-rm on redhat linux

I'm not on ubuntu

my system is following:

Linux ws-securehst02 2.6.32-504.el6.x86_64 #1 SMP Tue Sep 16 01:56:35 EDT 2014 x86_64 x86_64 x86_64 GNU/Linux

Revision history for this message
François Marier (fmarier) wrote :

Apologies for totally missing your bug report.

I'm not able to reproduce this on Ubuntu, but I do however note that the following is problematic:

1. put `/tmp/foo/` in ~/.config/safe-rm
2. try deleting that directory using `rm -rf /tmp/foo` or `rm -rf /tmp/foo/`

My expectation is that #2 would fail and the directory would be preserved, but it's not. A trailing slash means that the directory is fair game for deletion.

If you are still seeing something else on Red Hat, would you mind sharing a ~/.config/safe-rm config file which demonstrates the crash?

Changed in safe-rm:
importance: Undecided → Medium
description: updated
summary: - trailing slash in .config/safe-rm crashes safe-rm on redhat linux
+ Trailing slash in .config/safe-rm don't work as expected
Changed in safe-rm:
status: New → Confirmed
Changed in safe-rm:
milestone: none → 1.0.0
status: Confirmed → Fix Committed
tags: removed: slash trailing
Changed in safe-rm:
assignee: nobody → François Marier (fmarier)
Changed in safe-rm:
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.