rsnapshot LVM backup fails at line 3720

Bug #1270686 reported by Jon Stumpf
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
rsnapshot (Ubuntu)
Confirmed
Undecided
Unassigned

Bug Description

This error is received for each LVM listed in the rsnapshot.conf file:
        Use of uninitialized value $linux_lvm_oldpwd in concatenation (.) or string at /usr/bin/rsnapshot line 3720.
        chdir()
        Use of uninitialized value $linux_lvm_oldpwd in chdir at /usr/bin/rsnapshot line 3722.
        Use of chdir('') or chdir(undef) as chdir() is deprecated at /usr/bin/rsnapshot line 3722.

The error actually occurs at the assignment of $linux_lvm_oldpwd on line 3543.
        $linux_lvm_oldpwd = $ENV{PWD};

$ENV{PWD} returns nothing. This was not the case in December and this started failing for me sometime around Jan 10th, 2014. I am not a Perl programmer but the documentation I found states that the proper way to get the current working directory is to call cwd() and NOT use $ENV{PWD}.

I replaced the offending line with the following:
        $linux_lvm_oldpwd = cwd();

I no longer experience the issue.

- jss

Tags: patch
Revision history for this message
Jon Stumpf (jon-stumpf) wrote :
Revision history for this message
Jon Stumpf (jon-stumpf) wrote :

$ dpkg --status perl
Package: perl
Status: install ok installed
Priority: standard
Section: perl
Installed-Size: 17115
Maintainer: Ubuntu Developers <email address hidden>
Architecture: amd64
Version: 5.14.2-21build1
... lines deleted ...
$

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "rsnapshot.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in rsnapshot (Ubuntu):
status: New → Confirmed
Revision history for this message
Matthew Spraggs (matthew-spraggs) wrote :

Can confirm this bug, and that the proposed fix works.

Revision history for this message
Guillaume Delacour (gui-iroqwa) wrote :

This is fixed in Debian rsnapshot 1.4.0-1 package.

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.