22/970e5bd5e8e53ed56803a01625b970a2ab08b6

   1 Return-Path: <jani@nikula.org>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id 567C9431FB6\r
   6         for <notmuch@notmuchmail.org>; Thu,  5 Apr 2012 04:49:50 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id r8Ct+56Siksd for <notmuch@notmuchmail.org>;\r
  16         Thu,  5 Apr 2012 04:49:49 -0700 (PDT)\r
  17 Received: from mail-qc0-f181.google.com (mail-qc0-f181.google.com\r
  18         [209.85.216.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id B9F56431FAE\r
  21         for <notmuch@notmuchmail.org>; Thu,  5 Apr 2012 04:49:49 -0700 (PDT)\r
  22 Received: by qcsk26 with SMTP id k26so914560qcs.26\r
  23         for <notmuch@notmuchmail.org>; Thu, 05 Apr 2012 04:49:48 -0700 (PDT)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=google.com; s=20120113;\r
  26         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
  27         :mime-version:content-type:x-gm-message-state;\r
  28         bh=WIL/pphOGlbGKX82FJX6p/HT1+uR88ccUaHaFWRZ+lE=;\r
  29         b=XU/4/28o3mFB6WAz1nw/NUxs3ZIsvqkLiVb11q0/WGo/b4GBVtZrxCHevjnIcjKBVc\r
  30         SKUDmj+Rknl/XpHXtzbe47tBf7nRDbC/14Plvem8K1Wvjl8+BlBlMOaRg3DQsFVLVwa6\r
  31         +nlY5WhqCSPH0Xtr7MVutObFQ4VD4iZqzLVlT4ulCIqJDoltaJTui37xngZkYCoLe+Qh\r
  32         5/fp2CgM+4GrqR0nNle9xYeDn6z9bQ5XOOM7uR+GBJ7c7/oEkARyVu07QncfmzG98EqJ\r
  33         gjoIw3NRcss17YeCtPVDvDSMowcuCi6i6znzSJnLNzv5MLltn6sQl3JOTXYc+az/lUXE\r
  34         h+XA==\r
  35 Received: by 10.224.209.74 with SMTP id gf10mr3698421qab.8.1333626588029;\r
  36         Thu, 05 Apr 2012 04:49:48 -0700 (PDT)\r
  37 Received: from localhost (nikula.org. [92.243.24.172])\r
  38         by mx.google.com with ESMTPS id dv7sm5777813qab.15.2012.04.05.04.49.44\r
  39         (version=SSLv3 cipher=OTHER); Thu, 05 Apr 2012 04:49:46 -0700 (PDT)\r
  40 From: Jani Nikula <jani@nikula.org>\r
  41 To: David Bremner <david@tethera.net>, notmuch@notmuchmail.org\r
  42 Subject: Re: [PATCH 2/8] hex-escape: be more strict about the format while\r
  43         decoding\r
  44 In-Reply-To: <87fwcitnuk.fsf@zancas.localnet>\r
  45 References: <cover.1333231401.git.jani@nikula.org>\r
  46         <81e543344ffe8e2761afd57a2268e8b362f4aef4.1333231401.git.jani@nikula.org>User-Agent:    Notmuch/0.12+70~g46e73fe (http://notmuchmail.org) Emacs/23.3.1\r
  47         (x86_64-pc-linux-gnu) <87fwcitnuk.fsf@zancas.localnet>\r
  48 User-Agent: Notmuch/0.11.1+222~ga47a98c (http://notmuchmail.org) Emacs/23.1.1\r
  49         (i686-pc-linux-gnu)\r
  50 Date: Thu, 05 Apr 2012 11:49:42 +0000\r
  51 Message-ID: <87hawyxusp.fsf@nikula.org>\r
  52 MIME-Version: 1.0\r
  53 Content-Type: text/plain; charset=us-ascii\r
  54 X-Gm-Message-State:\r
  55  ALoCoQk4hl1VmUgwkmYyXD8wUAov4w10HLFgYJkUmk4x1+FtC/HdFpRW1Y0OtpMeCvOlyrwYW7bX\r
  56 X-BeenThere: notmuch@notmuchmail.org\r
  57 X-Mailman-Version: 2.1.13\r
  58 Precedence: list\r
  59 List-Id: "Use and development of the notmuch mail system."\r
  60         <notmuch.notmuchmail.org>\r
  61 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  62         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  63 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  64 List-Post: <mailto:notmuch@notmuchmail.org>\r
  65 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  66 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  67         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  68 X-List-Received-Date: Thu, 05 Apr 2012 11:49:50 -0000\r
  69 \r
  70 On Thu, 05 Apr 2012 08:33:23 -0300, David Bremner <david@tethera.net> wrote:\r
  71 > Jani Nikula <jani@nikula.org> writes:\r
  72 > \r
  73 > > Signed-off-by: Jani Nikula <jani@nikula.org>\r
  74 > >\r
  75 > > ---\r
  76 > >\r
  77 > > This could be folded to "hex-escape: (en|de)code strings to/from\r
  78 > > restricted character set".\r
  79 > \r
  80 > That's probably a good plan.\r
  81 > \r
  82 > > -       if (len < 3)\r
  83 > > +       if (!isxdigit ((unsigned char) p[1]) ||\r
  84 > > +           !isxdigit ((unsigned char) p[2]))\r
  85 > \r
  86 > What happens if there are not two characters after the escape? Is this\r
  87 > relying on calling isxdigit on the null terminator?\r
  88 \r
  89 It is, and technically there's nothing wrong with that. Would you prefer\r
  90 explicit checks for '\0' in the if condition, for clarity? Or a comment\r
  91 about it?\r
  92 \r
  93 Jani.\r