---
- branch: netbsd-8
  date: Mon Nov  6 09:37:24 UTC 2017
  files:
  - new: 1.5.22.1
    old: '1.5'
    path: src/external/bsd/nvi/dist/common/recover.c
    pathrev: src/external/bsd/nvi/dist/common/recover.c@1.5.22.1
    type: modified
  - new: 1.1.22.1
    old: '1.1'
    path: src/external/bsd/nvi/usr.bin/recover/virecover
    pathrev: src/external/bsd/nvi/usr.bin/recover/virecover@1.1.22.1
    type: modified
  id: 20171106T093724Z.2d9094438811d5087bcf541eefc0b3140a1e7592
  log: "Pull up following revision(s) (requested by spz in ticket #344):\n\texternal/bsd/nvi/usr.bin/recover/virecover:
    1.2-1.3\n\texternal/bsd/nvi/dist/common/recover.c: 1.6-1.9\nbe more careful about
    opening recovery files... in particular deal with\npeople trying to get 'vi -r'
    stuck using named pipes, symlink attacks,\nand coercing others opening recovery
    files they did not create.\n- don't use command substitution to glob a pattern
    into a list of filenames;\n  it is less efficient than doing it directly and does
    not handle whitespace\n  in filenames properly.\n- change test to [\n- quote variables\nPut
    back the tests for \"no files matched\" (in a different way than they\nwere written
    previously - but that's just style.)   This is not csh...\nUse the correct test
    operator to test for an empty file (rather than\ntesting for an empty file name...)\nWrite
    test ('[') commands in a way that is defined to work, rather than\njust happens
    to - we can afford the (negligible) performance hit here.\nDeal safely with recovery
    mail files.\noops, accidendally committed an earlier non-working version; fixed.\nDon't
    use popenve() for portability; forking an extra shell here is not an\nissue.\n"
  module: src
  subject: 'CVS commit: [netbsd-8] src/external/bsd/nvi'
  unixtime: '1509961044'
  user: snj