From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <zsh-workers-return-42534-mason-zsh=primenet.com.au@zsh.org>
Received: (qmail 9828 invoked by alias); 26 Mar 2018 16:45:33 -0000
Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm
Precedence: bulk
X-No-Archive: yes
List-Id: Zsh Workers List <zsh-workers.zsh.org>
List-Post: <mailto:zsh-workers@zsh.org>
List-Help: <mailto:zsh-workers-help@zsh.org>
List-Unsubscribe: <mailto:zsh-workers-unsubscribe@zsh.org>
X-Seq: 42534
Received: (qmail 3148 invoked by uid 1010); 26 Mar 2018 16:45:33 -0000
X-Qmail-Scanner-Diagnostics: from kahlil.inlv.org by f.primenet.com.au (envelope-from <martijn@inlv.org>, uid 7791) with qmail-scanner-2.11 
 (clamdscan: 0.99.2/21882. spamassassin: 3.4.1.  
 Clear:RC:0(37.59.109.123):SA:0(-1.9/5.0):. 
 Processed in 1.933844 secs); 26 Mar 2018 16:45:33 -0000
X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on f.primenet.com.au
X-Spam-Level: 
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,T_RP_MATCHES_RCVD
	autolearn=ham autolearn_force=no version=3.4.1
X-Envelope-From: martijn@inlv.org
X-Qmail-Scanner-Mime-Attachments: |
X-Qmail-Scanner-Zip-Files: |
From: Martijn Dekker <martijn@inlv.org>
To: Zsh hackers list <zsh-workers@zsh.org>
Subject: [PATCH] make sure internal temp files are user readable and writeable
Message-ID: <581009fe-372c-641a-cf5f-87e6a2a68dbb@inlv.org>
Date: Mon, 26 Mar 2018 18:45:23 +0200
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:52.0)
 Gecko/20100101 Thunderbird/52.6.0
MIME-Version: 1.0
Content-Type: multipart/mixed;
 boundary="------------D41EEAE5BC32912776D25414"
Content-Language: en-GB

--------------D41EEAE5BC32912776D25414
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

Here-documents fail if the umask does not allow user reading.

$ zsh -c $'umask u-r; cat <<EOF\nEOF'
zsh:1: can't create temp file for here document: permission denied

Silly though 'umask u-r' may be, the fact that here-documents use
temporary files on zsh is an internal implementation detail that should
make no difference to the script. It is trivial to save and temporarily
fix the umask during the creation of the temporary file.

This may fix a few other things that use gettempfile() as well.

- M.

--------------D41EEAE5BC32912776D25414
Content-Type: text/plain; charset=UTF-8; x-mac-type="0"; x-mac-creator="0";
 name="heredoc-umask.patch"
Content-Transfer-Encoding: base64
Content-Disposition: attachment;
 filename="heredoc-umask.patch"

ZGlmZiAtLWdpdCBhL1NyYy91dGlscy5jIGIvU3JjL3V0aWxzLmMKaW5kZXggNjUxN2UxNS4u
YTljNjIzMSAxMDA2NDQKLS0tIGEvU3JjL3V0aWxzLmMKKysrIGIvU3JjL3V0aWxzLmMKQEAg
LTIxNzcsNiArMjE3Nyw3IEBAIGdldHRlbXBmaWxlKGNvbnN0IGNoYXIgKnByZWZpeCwgaW50
IHVzZV9oZWFwLCBjaGFyICoqdGVtcG5hbWUpCiB7CiAgICAgY2hhciAqZm47CiAgICAgaW50
IGZkOworICAgIG1vZGVfdCBvbGRfdW1hc2sgPSB1bWFzaygwMTc3KTsKICNpZiBIQVZFX01L
U1RFTVAKICAgICBjaGFyICpzdWZmaXggPSBwcmVmaXggPyAiLlhYWFhYWCIgOiAiWFhYWFhY
IjsKIApAQCAtMjIxMiw2ICsyMjEzLDcgQEAgZ2V0dGVtcGZpbGUoY29uc3QgY2hhciAqcHJl
Zml4LCBpbnQgdXNlX2hlYXAsIGNoYXIgKip0ZW1wbmFtZSkKICNlbmRpZgogICAgICp0ZW1w
bmFtZSA9IGZuOwogCisgICAgdW1hc2sob2xkX3VtYXNrKTsKICAgICB1bnF1ZXVlX3NpZ25h
bHMoKTsKICAgICByZXR1cm4gZmQ7CiB9CmRpZmYgLS1naXQgYS9UZXN0L0EwNHJlZGlyZWN0
Lnp0c3QgYi9UZXN0L0EwNHJlZGlyZWN0Lnp0c3QKaW5kZXggZWY3ZGRiMi4uYjViNjVjZiAx
MDA2NDQKLS0tIGEvVGVzdC9BMDRyZWRpcmVjdC56dHN0CisrKyBiL1Rlc3QvQTA0cmVkaXJl
Y3QuenRzdApAQCAtNjY3LDMgKzY2NywxMiBAQAogMDpSZWRpcmVjdCBpbiB0aGUgbWlkZGxl
IG9mIGFzc2lnbm1lbnRzCiA+YgogPmQKKworICB1bWFzayAwNzc3CisgIGNhdCA8PCcgIEhF
UkUnCisgIGxvb2sgbWEsIG5vIHBlcm1pc3Npb25zCisgIEhFUkUKKyAgY2F0IDw8PCJpdCdz
IGEgbWlyYWNsZSIKKzA6SGVyZS17c3RyaW5nLGRvY3VtZW50fXMgc3VjY2VlZCB3aXRoIHJl
c3RyaWN0aXZlIHVtYXNrCis+ICBsb29rIG1hLCBubyBwZXJtaXNzaW9ucworPml0J3MgYSBt
aXJhY2xlCg==
--------------D41EEAE5BC32912776D25414--