smcv [Thu, 29 Dec 2016 20:24:48 +0000 (16:24 -0400)]
add anchors for use in advisory to oss-security
Simon McVittie [Thu, 29 Dec 2016 20:08:49 +0000 (20:08 +0000)]
Clarify which versions of ikiwiki fixed CVE-2016-9645, -9646
Simon McVittie [Thu, 29 Dec 2016 17:37:51 +0000 (17:37 +0000)]
Simon McVittie [Thu, 29 Dec 2016 17:31:30 +0000 (17:31 +0000)]
Add CVE references for CVE-2016-9646, CVE-2016-9645
Thanks to the Debian security team for allocating these.
Simon McVittie [Thu, 29 Dec 2016 17:30:56 +0000 (17:30 +0000)]
Prune git remotes that are unreachable or unresponsive
Simon McVittie [Wed, 28 Dec 2016 21:03:01 +0000 (21:03 +0000)]
git: Add test coverage for reverting attachments
Simon McVittie [Wed, 28 Dec 2016 19:35:14 +0000 (19:35 +0000)]
git: write proposed attachment to temp file without going via system()
Simon McVittie [Wed, 28 Dec 2016 19:26:33 +0000 (19:26 +0000)]
git: change calling convention of safe_git to have named arguments
Simon McVittie [Wed, 28 Dec 2016 18:04:34 +0000 (18:04 +0000)]
git: Do the revert operation in a secondary working tree
This avoids leaving the git directory in an inconsistent state if the
host system is rebooted while we are processing a revert.
Simon McVittie [Wed, 28 Dec 2016 18:02:59 +0000 (18:02 +0000)]
git: Turn $git_dir into a stack
This will be necessary when we use a secondary working tree to do
reverts without leaving the primary working tree in an inconsistent
state.
Simon McVittie [Sat, 24 Dec 2016 15:58:10 +0000 (15:58 +0000)]
Add automated test for using the CGI with git, including CVE-2016-10026
Simon McVittie [Mon, 19 Dec 2016 13:48:56 +0000 (13:48 +0000)]
Try revert operations (on a branch) before approving them
Otherwise, we have a time-of-check/time-of-use vulnerability:
rcs_preprevert previously looked at what changed in the commit we are
reverting, not at what would result from reverting it now. In
particular, if some files were renamed since the commit we are
reverting, a revert of changes that were within the designated
subdirectory and allowed by check_canchange() might now affect
files that are outside the designated subdirectory or disallowed
by check_canchange().
It is not sufficient to disable rename detection, since git older
than 2.8.0rc0 (in particular the version in Debian stable) silently
accepts and ignores the relevant options.
OVE-
20161226-0002
Simon McVittie [Mon, 26 Dec 2016 16:30:43 +0000 (16:30 +0000)]
Revert "Tell `git revert` not to follow renames"
This doesn't work prior to git 2.8: `git revert` silently ignores the
option and succeeds. We will have to fix CVE-2016-10026 some other way.
This reverts commit
9cada49ed6ad24556dbe9861ad5b0a9f526167f9.
Simon McVittie [Sat, 24 Dec 2016 15:03:51 +0000 (15:03 +0000)]
Force CGI::FormBuilder->field to scalar context where necessary
CGI::FormBuilder->field has behaviour similar to the CGI.pm misfeature
we avoided in
f4ec7b0. Force it into scalar context where it is used
in an argument list.
This prevents two (relatively minor) commit metadata forgery
vulnerabilities:
* In the comments plugin, an attacker who was able to post a comment
could give it a user-specified author and author-URL even if the wiki
configuration did not allow for that, by crafting multiple values
to other fields.
* In the editpage plugin, an attacker who was able to edit a page
could potentially forge commit authorship by crafting multiple values
for the rcsinfo field.
The remaining plugins changed in this commit appear to have been
protected by use of explicit scalar prototypes for the called functions,
but have been changed anyway to make them more obviously correct.
In particular, checkpassword() in passwordauth has a known prototype,
so an attacker cannot trick it into treating multiple values of the
name field as being the username, password and field to check for.
OVE-
20161226-0001
Simon McVittie [Sat, 24 Dec 2016 14:35:01 +0000 (14:35 +0000)]
git: do not fail to commit if committer is anonymous
Simon McVittie [Sat, 24 Dec 2016 14:34:30 +0000 (14:34 +0000)]
git: don't issue a warning if rcsinfo is undefined
The intention here seems to be that $prev may be undefined, and the
only way that can legitimately happen is for $params{token} to be
undefined too.
Simon McVittie [Sat, 24 Dec 2016 12:27:21 +0000 (12:27 +0000)]
git_revert test: reinstate ikiwiki.setup, and make it work uninstalled
Previously it was relying on running with an installed ikiwiki
and being able to copy in recentchanges.mdwn and wikiicons/ from the
underlay in /usr. The underlay in ./underlays/basewiki can't be used
(yet) because ikiwiki doesn't allow following symlinks, even from
underlays.
I'd like to make ikiwiki follow symlinks whose destinations can be
verified to be safe (for example making it willing to expose
/usr/share/javascript to the web, but not /etc/passwd), at least from
underlays, but this is security-sensitive so I'm not going to rush
into it.
spalax [Mon, 26 Dec 2016 22:03:28 +0000 (18:03 -0400)]
Added a comment
smcv [Mon, 26 Dec 2016 19:26:25 +0000 (15:26 -0400)]
Added a comment
spalax [Sun, 25 Dec 2016 21:05:08 +0000 (17:05 -0400)]
Question about default timezone ":/etc/localtime"
Simon McVittie [Wed, 21 Dec 2016 13:03:32 +0000 (13:03 +0000)]
Add CVE references for CVE-2016-10026
intrigeri [Tue, 20 Dec 2016 10:26:22 +0000 (10:26 +0000)]
Replied.
Simon McVittie [Mon, 19 Dec 2016 21:20:41 +0000 (21:20 +0000)]
Simon McVittie [Mon, 19 Dec 2016 20:35:01 +0000 (20:35 +0000)]
smcv [Mon, 19 Dec 2016 20:33:48 +0000 (16:33 -0400)]
mention security contacts here too
Amitai Schleier [Mon, 19 Dec 2016 19:23:43 +0000 (20:23 +0100)]
Opt in to whatever spam this may bring.
Simon McVittie [Mon, 19 Dec 2016 17:29:27 +0000 (17:29 +0000)]
changelog
intrigeri [Sat, 19 Nov 2016 19:20:48 +0000 (19:20 +0000)]
Make pagestats output more deterministic.
Sort in lexical order the pages that have the same number of hits.
Simon McVittie [Mon, 19 Dec 2016 17:25:35 +0000 (17:25 +0000)]
Update changelog
Simon McVittie [Mon, 19 Dec 2016 16:59:43 +0000 (16:59 +0000)]
Restrict CSS matches on .header to not affect <tr>
Pandoc generates <tr class="header"> to hold <th> elements, and
we don't want to make those be display: block.
Signed-off-by: Simon McVittie <smcv@debian.org>
Simon McVittie [Mon, 19 Dec 2016 16:23:54 +0000 (16:23 +0000)]
List security contacts
We still don't have a security@ alias; listing personal emails is
unfortunately the next-best thing.
Simon McVittie [Mon, 19 Dec 2016 15:33:28 +0000 (15:33 +0000)]
Add a manual test for reverting git commits
Signed-off-by: Simon McVittie <smcv@debian.org>
Simon McVittie [Mon, 19 Dec 2016 13:48:56 +0000 (13:48 +0000)]
Tell `git revert` not to follow renames
Otherwise, we have an authorization bypass vulnerability: rcs_preprevert
looks at what changed in the commit we are reverting, not at what would
result from reverting it now. In particular, if some files were renamed
since the commit we are reverting, a revert of changes that were within
the designated subdirectory and allowed by check_canchange() might now
affect files that are outside the designated subdirectory or disallowed
by check_canchange().
Signed-off-by: Simon McVittie <smcv@debian.org>
smcv [Mon, 19 Dec 2016 17:23:06 +0000 (13:23 -0400)]
Added a comment: no, not supported
smcv [Mon, 19 Dec 2016 16:46:46 +0000 (12:46 -0400)]
rename bugs/img_tag_should_support_relative_size.mdwn to todo/img_tag_should_support_relative_size.mdwn
smcv [Mon, 19 Dec 2016 16:46:22 +0000 (12:46 -0400)]
Not possible as stated, but could be adapted into a valid feature request
smcv [Mon, 19 Dec 2016 13:55:58 +0000 (09:55 -0400)]
Simon McVittie [Mon, 19 Dec 2016 12:00:34 +0000 (12:00 +0000)]
cgitemplate: actually remove dead code
Signed-off-by: Simon McVittie <smcv@debian.org>
Simon McVittie [Mon, 19 Dec 2016 12:00:34 +0000 (12:00 +0000)]
cgitemplate: remove dead code
blipvert points out in [[bugs/use of $topurl in cgitemplate]] that this
variable has not been used since commit
a052771
"Now that we're always using HTML5, <base href> can be relative".
Signed-off-by: Simon McVittie <smcv@debian.org>
intrigeri [Sat, 17 Dec 2016 11:11:44 +0000 (11:11 +0000)]
Report authorization bypass via RCS revert.
intrigeri [Sun, 20 Nov 2016 07:00:20 +0000 (07:00 +0000)]
pagestats determinism: report bug + patch.
svetlana.nfshost
Juego [Wed, 16 Nov 2016 22:17:48 +0000 (18:17 -0400)]
Added custom solution
Juego [Wed, 16 Nov 2016 22:15:14 +0000 (18:15 -0400)]
rename forum/FastCGI_problem_on_Arch.mdwn to forum/__91__Solved__93__FastCGI_problem_on_Arch.mdwn
Amitai Schleier [Sun, 13 Nov 2016 03:02:58 +0000 (22:02 -0500)]
Update my personal site URL.
james@2468840dc8f314e837e1fde99a5fb1b884fa993a [Sun, 13 Nov 2016 00:08:40 +0000 (20:08 -0400)]
update my site links.
openmedi [Thu, 10 Nov 2016 17:09:41 +0000 (13:09 -0400)]
Added a comment
openmedi [Thu, 10 Nov 2016 17:06:23 +0000 (13:06 -0400)]
Added a comment
openmedi [Thu, 10 Nov 2016 17:03:00 +0000 (13:03 -0400)]
openmedi [Sun, 6 Nov 2016 19:36:24 +0000 (15:36 -0400)]
Added a comment
openmedi [Thu, 3 Nov 2016 22:13:15 +0000 (18:13 -0400)]
Added a comment
https://id.koumbit.net/anarcat [Tue, 1 Nov 2016 15:56:18 +0000 (11:56 -0400)]
consider portier as a successor to OpenID?
https://id.koumbit.net/anarcat [Tue, 1 Nov 2016 15:55:46 +0000 (11:55 -0400)]
introduce portier here as well, while i'm here
https://id.koumbit.net/anarcat [Tue, 1 Nov 2016 15:49:48 +0000 (11:49 -0400)]
nextgen persona?
https://id.koumbit.net/anarcat [Tue, 1 Nov 2016 15:45:31 +0000 (11:45 -0400)]
another look at bootstrap and packaging strategies
Amitai Schleier [Mon, 24 Oct 2016 01:00:36 +0000 (21:00 -0400)]
The C2 wiki appears to have moved.
openmedi [Sun, 16 Oct 2016 16:38:47 +0000 (12:38 -0400)]
icydee [Fri, 7 Oct 2016 11:08:35 +0000 (07:08 -0400)]
karsk [Fri, 30 Sep 2016 08:10:10 +0000 (04:10 -0400)]
That was a (curious) mistake.
This reverts commit
1bfe2e2e19bf45bac52c0cc0bc0b17cea64887b6
karsk [Fri, 30 Sep 2016 08:09:12 +0000 (04:09 -0400)]
removed
spalax [Tue, 27 Sep 2016 19:08:30 +0000 (15:08 -0400)]
Added a comment: Translating "Last edited"
karsk [Tue, 27 Sep 2016 13:56:51 +0000 (09:56 -0400)]
karsk [Tue, 27 Sep 2016 13:22:47 +0000 (09:22 -0400)]
karsk [Tue, 27 Sep 2016 12:41:28 +0000 (08:41 -0400)]
alexjj [Sun, 25 Sep 2016 04:51:03 +0000 (00:51 -0400)]
added actual progress bar
alexjj [Sat, 24 Sep 2016 06:42:34 +0000 (02:42 -0400)]
alexjj [Sat, 24 Sep 2016 06:39:10 +0000 (02:39 -0400)]
alexjj [Sat, 24 Sep 2016 05:58:07 +0000 (01:58 -0400)]
removed
alexjj [Sat, 24 Sep 2016 05:53:46 +0000 (01:53 -0400)]
alexjj [Sat, 24 Sep 2016 04:48:51 +0000 (00:48 -0400)]
Created
alexjj [Sat, 24 Sep 2016 04:22:08 +0000 (00:22 -0400)]
alexjj [Sat, 24 Sep 2016 04:17:05 +0000 (00:17 -0400)]
formatting
alexjj [Sat, 24 Sep 2016 04:16:00 +0000 (00:16 -0400)]
modern nginx settings suggestion
Joey Hess [Wed, 21 Sep 2016 17:51:42 +0000 (13:51 -0400)]
inline: Prevent creating a file named ".mdwn" when the postform is submitted with an empty title.
Joey Hess [Wed, 21 Sep 2016 17:44:57 +0000 (13:44 -0400)]
bug
alexjj@97b75209148c043997fe05b4341a629090820035 [Tue, 20 Sep 2016 16:59:27 +0000 (12:59 -0400)]
Added a comment: ever fix this?
alexjj@97b75209148c043997fe05b4341a629090820035 [Tue, 20 Sep 2016 16:05:54 +0000 (12:05 -0400)]
added remark to nginx
alexjj@97b75209148c043997fe05b4341a629090820035 [Tue, 20 Sep 2016 16:04:29 +0000 (12:04 -0400)]
Created
http://pnijjar.livejournal.com/ [Sun, 18 Sep 2016 05:40:20 +0000 (01:40 -0400)]
Amitai Schlair [Wed, 14 Sep 2016 18:32:03 +0000 (14:32 -0400)]
Rename this redirect page, missed in previous.
Amitai Schlair [Wed, 14 Sep 2016 18:28:01 +0000 (14:28 -0400)]
Update my surname to its new legal spelling.
simonmic [Sun, 11 Sep 2016 18:35:37 +0000 (14:35 -0400)]
sign comment
simonmic [Sun, 11 Sep 2016 18:34:33 +0000 (14:34 -0400)]
fix link
simonmic [Sun, 11 Sep 2016 17:39:32 +0000 (13:39 -0400)]
update
simonmic [Sun, 11 Sep 2016 17:33:23 +0000 (13:33 -0400)]
minor edit
simonmic [Sun, 11 Sep 2016 17:32:39 +0000 (13:32 -0400)]
holger [Wed, 7 Sep 2016 12:20:55 +0000 (08:20 -0400)]
holger [Wed, 7 Sep 2016 12:17:16 +0000 (08:17 -0400)]
cleaned up, updated and submitted for patch
Simon McVittie [Tue, 6 Sep 2016 17:12:54 +0000 (18:12 +0100)]
Belatedly announce yesterday's release
Simon McVittie [Mon, 5 Sep 2016 20:26:32 +0000 (21:26 +0100)]
Simon McVittie [Sat, 3 Sep 2016 22:46:33 +0000 (23:46 +0100)]
pm_filter: use \Q...\E to escape all possible strings pedantically
The current implementation would misbehave for prefixes containing
a single quote.
Simon McVittie [Sat, 3 Sep 2016 22:29:37 +0000 (23:29 +0100)]
Use git log --no-renames for recentchanges
Otherwise, recent git releases show renames as renames, and we do not
see that newdir/test5 was affected.
Bug-Debian: https://bugs.debian.org/835612
https://id.koumbit.net/anarcat [Tue, 30 Aug 2016 03:51:53 +0000 (23:51 -0400)]
guh, then i mess up the markdown, how ironic... btw, commonmark parsed my original version correctly :p
https://id.koumbit.net/anarcat [Tue, 30 Aug 2016 03:50:26 +0000 (23:50 -0400)]
clarify and more links