Bug#276754: patch

[Colin Watson <cjwatson@debian.org>]

On Sat, Oct 23, 2004 at 10:21:31PM +0100, Mark Hymers wrote:
> Apologies, I screwed up by sending the patch to control twice.
> No idea why I did that.....  Here it is again:
> 
> 
> Here's a patch to fix this bug.  Tested by upgrading a woody chroot to
> the current version without and with the patch.
[...]
> diff -u openssh-3.8.1p1/debian/postinst openssh-3.8.1p1/debian/postinst
> --- openssh-3.8.1p1/debian/postinst
> +++ openssh-3.8.1p1/debian/postinst
> @@ -13,7 +13,16 @@
>    exit 0
>  fi
>  
> -
> +# This routine takes two file names, copies the first to the
> +# second ensuring that permissions are maintained ignoring umask
> +copyfile() {
> +	basefile="$1"
> +	targetfile="$2"
> +	existingumask="$(umask)"
> +	umask 0000
> +	cp -a "$basefile" "$targetfile"
> +	umask "$existingumask"
> +}

Thanks for the patch. I have a much, much simpler one in my working
copy, though; all that's needed is to use chown --reference and chmod
--reference when setting configuration options. I hope to test and
upload this tomorrow.

Cheers,

-- 
Colin Watson                                       [cjwatson@debian.org]