Merge lp:~ritusparks/ubuntu/precise/secure-delete/bug-1213373 into lp:ubuntu/precise/secure-delete

Proposed by Ritu
Status: Needs review
Proposed branch: lp:~ritusparks/ubuntu/precise/secure-delete/bug-1213373
Merge into: lp:ubuntu/precise/secure-delete
Diff against target: 205 lines (+41/-28)
10 files modified
debian/changelog (+18/-0)
debian/control (+3/-4)
debian/rules (+9/-10)
debian/source/format (+1/-0)
debian/watch (+0/-4)
sfill.1 (+2/-2)
smem.1 (+3/-3)
srm.1 (+3/-3)
srm.c (+1/-1)
sswap.1 (+1/-1)
To merge this branch: bzr merge lp:~ritusparks/ubuntu/precise/secure-delete/bug-1213373
Reviewer Review Type Date Requested Status
Marc Deslauriers Disapprove
Review via email: mp+193236@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

Thanks for the merge proposal.

I am going to have to NACK this. Not only is this backporting a bunch of unrelated changes to Precise, but I don't think a typo in a man page is enough of a reason to qualify for an SRU.

See the criteria here:
https://wiki.ubuntu.com/StableReleaseUpdates

Thanks for your contribution.

review: Disapprove

Unmerged revisions

6. By Ritu

The man page of sfill had a 'typo,' the pural of thief had been spelt as thiefs instead of thieves

5. By Robert Lemmen <email address hidden>

* Enable hardening flags during build
* Update packaging
* Bump standards-version
* Fix a few minus-vs-hyphen bugs in manpages
* Fixed a few typos (closes: #486018, #653260)
* Tone down description a little bit and include binary names
  (closes: #680336, #538151)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2010-01-05 14:55:00 +0000
3+++ debian/changelog 2013-10-30 13:24:49 +0000
4@@ -1,3 +1,21 @@
5+secure-delete (3.1-6ubuntu1) precise; urgency=low
6+
7+The man page of sfill had a typo. The pural of theif was misspelt as theifs instead of thieves *
8+
9+ -- Ritu Sreekumaran <ritusparks@gmail.com> Wed, 30 Oct 2013 17:26:23 +0530
10+
11+secure-delete (3.1-6) unstable; urgency=low
12+
13+ * Enable hardening flags during build
14+ * Update packaging
15+ * Bump standards-version
16+ * Fix a few minus-vs-hyphen bugs in manpages
17+ * Fixed a few typos (closes: #486018, #653260)
18+ * Tone down description a little bit and include binary names
19+ (closes: #680336, #538151)
20+
21+ -- Robert Lemmen <robertle@semistable.com> Fri, 05 Jul 2013 08:50:37 +0100
22+
23 secure-delete (3.1-5) unstable; urgency=low
24
25 * Rename smem to sdmem to avoid name clash with smem package
26
27=== modified file 'debian/control'
28--- debian/control 2010-01-05 14:55:00 +0000
29+++ debian/control 2013-10-30 13:24:49 +0000
30@@ -3,12 +3,11 @@
31 Priority: optional
32 Maintainer: Robert Lemmen <robertle@semistable.com>
33 Build-Depends: debhelper (>= 7.0.0)
34-Standards-Version: 3.8.3
35+Standards-Version: 3.9.4
36
37 Package: secure-delete
38 Architecture: any
39 Depends: ${shlibs:Depends}, ${misc:Depends}
40 Description: tools to wipe files, free disk space, swap and memory
41- Even if you overwrite a file 10+ times, it can still be recovered. This
42- package contains tools to securely wipe data from files, free disk space,
43- swap and memory.
44+ Gutmann method based tools for securely wiping data from files, free
45+ disk space, swap and memory: srm, sfill, sswap and sdmem.
46
47=== modified file 'debian/rules'
48--- debian/rules 2010-01-05 14:55:00 +0000
49+++ debian/rules 2013-10-30 13:24:49 +0000
50@@ -3,19 +3,18 @@
51 # Uncomment this to turn on verbose mode.
52 #export DH_VERBOSE=1
53
54-CFLAGS = -Wall -g
55-
56-ifneq (,$(findstring noopt,$(DEB_BUILD_OPTIONS)))
57- CFLAGS += -O0
58-else
59- CFLAGS += -O2
60-endif
61-
62-build: build-stamp
63+CPPFLAGS:=$(shell dpkg-buildflags --get CPPFLAGS)
64+CFLAGS:=$(shell dpkg-buildflags --get CFLAGS)
65+CXXFLAGS:=$(shell dpkg-buildflags --get CXXFLAGS)
66+LDFLAGS:=$(shell dpkg-buildflags --get LDFLAGS)
67+
68+build: build-arch build-indep
69+build-arch: build-stamp
70+build-indep: build-stamp
71
72 build-stamp:
73 dh_testdir
74- CFLAGS="$(CFLAGS)" $(MAKE)
75+ CFLAGS="$(CFLAGS) $(CPPFLAGS) $(LDFLAGS)" $(MAKE)
76 touch build-stamp
77
78 clean:
79
80=== added directory 'debian/source'
81=== added file 'debian/source/format'
82--- debian/source/format 1970-01-01 00:00:00 +0000
83+++ debian/source/format 2013-10-30 13:24:49 +0000
84@@ -0,0 +1,1 @@
85+1.0
86
87=== modified file 'debian/watch'
88--- debian/watch 2006-10-05 22:04:17 +0000
89+++ debian/watch 2013-10-30 13:24:49 +0000
90@@ -1,7 +1,3 @@
91-# Example watch control file for uscan
92-# Rename this file to "watch" and then you can run the "uscan" command
93-# to check for upstream updates and more.
94-# Site Directory Pattern Version Script
95 version=2
96 http://www.thc.org/releases.php .*secure[-_]delete-(.*)\.tar\.gz
97
98
99=== modified file 'sfill.1'
100--- sfill.1 2010-01-05 14:55:00 +0000
101+++ sfill.1 2013-10-30 13:24:49 +0000
102@@ -16,7 +16,7 @@
103 is designed to delete data which lies on available diskspace on mediums
104 in a secure manner which can not be recovered by thiefs, law enforcement
105 or other threats.
106-The wipe algorythm is based on the paper "Secure Deletion of Data from
107+The wipe algorithm is based on the paper "Secure Deletion of Data from
108 Magnetic and Solid-State Memory" presented at the 6th Usenix Security
109 Symposium by Peter Gutmann, one of the leading civilian cryptographers.
110 .PP
111@@ -59,7 +59,7 @@
112 and a final mode with random values.
113 .TP
114 .B \-l
115--l for a second time lessons the security even more: only one random pass
116+\-l for a second time lessons the security even more: only one random pass
117 is written.
118 .TP
119 .B \-v
120
121=== modified file 'smem.1'
122--- smem.1 2010-01-05 14:55:00 +0000
123+++ smem.1 2013-10-30 13:24:49 +0000
124@@ -51,7 +51,7 @@
125 and a final random one.
126 .TP
127 .B \-l
128--l for a second time lessons the security even more: only one pass with
129+\-l for a second time lessons the security even more: only one pass with
130 0x00 is written.
131 .TP
132 .B \-v
133@@ -61,13 +61,13 @@
134 .SH NOTE
135 .TP
136 This utility was originally called
137-.I sdmem
138+.I smem
139 but was renamed for debian to avoid name clashes with another package.
140
141 .SH BEWARE
142 .TP
143 .B SLOW
144-Wiping the memory is very slow. You might use sdmem with the -ll option. (tip)
145+Wiping the memory is very slow. You might use sdmem with the \-ll option. (tip)
146 .TP
147 .B BETA!
148 .I sdmem
149
150=== modified file 'srm.1'
151--- srm.1 2010-01-05 14:55:00 +0000
152+++ srm.1 2013-10-30 13:24:49 +0000
153@@ -55,7 +55,7 @@
154 .TP
155 .B \-d
156 ignore the two special dot files . and .. on the commandline. (so you can
157-execute it like "srm -d .* *")
158+execute it like "srm \-d .* *")
159 .TP
160 .B \-f
161 fast (and insecure mode): no /dev/urandom, no synchronize mode.
162@@ -65,7 +65,7 @@
163 and a final mode random values.
164 .TP
165 .B \-l
166--l for a second time lessons the security even more: only one random pass
167+\-l for a second time lessons the security even more: only one random pass
168 is written.
169 .TP
170 .B \-r
171@@ -95,7 +95,7 @@
172 which comes with the
173 .I secure_deletion package
174 to ensure to wipe also the free diskspace. However, If already a small
175-file aquired a block with your precious data, no tool known to me can help
176+file acquired a block with your precious data, no tool known to me can help
177 you here. For a secure deletion of the swap space
178 .I sswap
179 is available.
180
181=== modified file 'srm.c'
182--- srm.c 2010-01-05 14:55:00 +0000
183+++ srm.c 2013-10-30 13:24:49 +0000
184@@ -110,7 +110,7 @@
185 if ( (filestat.st_dev != controlstat.st_dev) || (filestat.st_ino != controlstat.st_ino) ) {
186 fprintf(stderr, "Race found! (directory %s became a link)\n", filename);
187 } else {
188- if ((dir = opendir (".")) != NULL) {
189+ if ((dir = opendir (".")) < 0) {
190 (void) chmod(".", 0700); /* ignore permission errors */
191 dir = opendir (".");
192 }
193
194=== modified file 'sswap.1'
195--- sswap.1 2010-01-05 14:55:00 +0000
196+++ sswap.1 2013-10-30 13:24:49 +0000
197@@ -49,7 +49,7 @@
198 a final mode with random values.
199 .TP
200 .B \-l
201--l for a second time lessons the security even more: only one pass with
202+\-l for a second time lessons the security even more: only one pass with
203 random values is written.
204 .TP
205 .B \-v

Subscribers

People subscribed via source and target branches

to all changes: