From owner-freebsd-hackers@freebsd.org  Sun Nov  5 12:05:32 2017
Return-Path: <owner-freebsd-hackers@freebsd.org>
Delivered-To: freebsd-hackers@mailman.ysv.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org
 [IPv6:2001:1900:2254:206a::19:1])
 by mailman.ysv.freebsd.org (Postfix) with ESMTP id D58A5E69BEF
 for <freebsd-hackers@mailman.ysv.freebsd.org>;
 Sun,  5 Nov 2017 12:05:32 +0000 (UTC) (envelope-from yuripv@gmx.com)
Received: from mout.gmx.net (mout.gmx.net [212.227.17.22])
 (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits))
 (Client CN "mout.gmx.net", Issuer "TeleSec ServerPass DE-2" (verified OK))
 by mx1.freebsd.org (Postfix) with ESMTPS id 3C7A47D151
 for <freebsd-hackers@freebsd.org>; Sun,  5 Nov 2017 12:05:32 +0000 (UTC)
 (envelope-from yuripv@gmx.com)
Received: from [192.168.1.2] ([94.233.224.52]) by mail.gmx.com (mrgmx101
 [212.227.17.174]) with ESMTPSA (Nemesis) id 0Lat5o-1dQwFN0wBM-00kLeB; Sun, 05
 Nov 2017 13:05:29 +0100
Subject: Re: "pw useradd -D -e" writes bogus expire_days value into pw.conf
From: Yuri Pankov <yuripv@gmx.com>
To: Victor Sudakov <vas@mpeks.tomsk.su>, freebsd-hackers@freebsd.org
References: <20171105033222.GA34641@admin.sibptus.transneft.ru>
 <497adca8-9e9a-fa99-34cb-6073cd433d01@gmx.com>
Message-ID: <1b21bd92-ae86-e81b-cf00-b2f7d9e06df7@gmx.com>
Date: Sun, 5 Nov 2017 15:05:28 +0300
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101
 Thunderbird/52.4.0
MIME-Version: 1.0
In-Reply-To: <497adca8-9e9a-fa99-34cb-6073cd433d01@gmx.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
X-Provags-ID: V03:K0:Q+U1CSn5MkC4wNbUe/YzSYzYgb0TZClX70+HLmnosj3Yf4vr/u0
 1BILJy8IxetJTQqn66gK+fjdLbreXM5Wcbc8u7e76THdL3HJs2RNcvPdjpQgbV0diFnOBS/
 G5RZK7UzzrGb7qEzUis3NhJXmYEi1GW747v2aEbgZvp8wXorPVn14NCYvSZH9iM4qVtn6xO
 9qaKBinPoc9aU6dTDgVnQ==
X-UI-Out-Filterresults: notjunk:1;V01:K0:ZZ0nh0nWFmg=:/DyY1Kl+koes694SKXI4VV
 tOsWE/VV2xmDYu9/z2C1V8pC98VKbIFaBr9U9gYtJf3KCRrq/EtSo6+lBRXMpwOkv+b7g6kTK
 yIyh3zW3eCKhfyLmvVzpWTP6N2cxStgI7Egx1nRo23Sea69n7GdaLpkaqn5vshkogpwQYRZ7I
 47RQrKWcqjmoqcYaxwnKvo9txAk+yXZCV1JMvC5XrnAmlnn/h9ZYOu7Bryxu+32yWtnqgAcjd
 xRcBeEcLMBuxqUX3oYP3k6GFR+a1ojN29YTqGweVWBfDnFRhk7BHSoUxraJuwjfMvX8jHQQFw
 HuAXHrPbLGOkAIIq3QZZIaKjQMaoHAQNV6Sv4Pfh/8rkLBC7tJwIn2A8vKADiRDERb0HUUauh
 iXulXwQ1BpQrZhu4ib/VdoWrFxras5JQlAE3VRLahUdrj2k41oG8r3Mmv1dcSBYQO9B7+o6Y0
 Evtyeo39PbzHej2GEBOjdQJSv33DNDU1hOjNgfieNmn0EPpANWJr4Sx9JDV5ujYxLNnwHH4Da
 //w0R/9O5/BzzDvGPWsyAwE5jAH93orRt+l4s4XdHu9YrO9iMX7YV6O1ykNGlE0Wl9YXdGUDA
 ppKef8r61q6rj6HxopQ5dJFa+XeKW5/8tl5OrVPf/2pRd7NjOvvidkqhudvPSpgpjUJ7SZ5AM
 Mb1wnqyNm0AxKIwswKA3RQice1gNvhfsXMB5I94mfjunJA62Ky1YMySIqclAYd6SX1lgoX40g
 horcslyx2zJLGr0UEqSra5fVnaWN5kD7XU2pqyKW1hKkax+sSG5vqomS9lxILwQkezLzGtkqU
 dj9PruBdCKuy0nXoAmrGjoEAi3sEnmBGadEbMCFDjC7LDXLMM68up2AndFt4YqNUkKHF68/
X-BeenThere: freebsd-hackers@freebsd.org
X-Mailman-Version: 2.1.23
Precedence: list
List-Id: Technical Discussions relating to FreeBSD
 <freebsd-hackers.freebsd.org>
List-Unsubscribe: <https://lists.freebsd.org/mailman/options/freebsd-hackers>, 
 <mailto:freebsd-hackers-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-hackers/>
List-Post: <mailto:freebsd-hackers@freebsd.org>
List-Help: <mailto:freebsd-hackers-request@freebsd.org?subject=help>
List-Subscribe: <https://lists.freebsd.org/mailman/listinfo/freebsd-hackers>, 
 <mailto:freebsd-hackers-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 05 Nov 2017 12:05:32 -0000

On Sun, 5 Nov 2017 14:45:40 +0300, Yuri Pankov wrote:
> On Sun, 5 Nov 2017 10:32:22 +0700, Victor Sudakov wrote:
>> Dear Colleagues,
>>
>> Could anyone please reproduce this bug, especially on an 11.x and 12.x
>> branches? Or is it just me?
>>
>> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=223431
> 
> Yes, it's the same on -CURRENT, but as the man page says, argument to -e
> must be numeric in presence of -D, so most likely it just needs an
> additional check.

Added a simple patch to the PR -- check if argument is numeric, 
otherwise it's parsed as date, and the *timestamp* of account expiry is 
written as days into the pw.conf.