[PATCH] use tdbbackup, not shutil.copy, skip 'upgrade' of privileges.ldb

[Michael Wood]

On 17 February 2013 10:39, Michael Wood <esiotrot at gmail.com> wrote:
> Hi
>
> On 17 February 2013 10:09, Andrew Bartlett <abartlet at samba.org> wrote:
>> This series of patches (in on and top of my upgradeprovision branch)
>> makes an important fix to the upgrade code, because calling shutil.copy
>> is NOT A SAFE WAY to deal with tdb (indeed any) databases.
>
> I thought shutil.copy would be perfectly safe as long as Samba was
> shut down at the time.  Is this not the case?

Never mind.  I've just realised that upgradeprovision would be
accessing tdb files, so shutil.copy does seem suspect, unless it does
the copies in such a way that there's no chance that any tdb files are
open at the time.

-- 
Michael Wood <esiotrot at gmail.com>