Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit7d7435c

Browse files
committed
Make current_logfiles use permissions assigned to files in data directory
Since its introduction in19dc233, current_logfiles has been assignedthe same permissions as a log file, which can be enforced withlog_file_mode. This setup can lead to incompatibility problems withgroup access permissions as current_logfiles is not located in the logdirectory, but at the root of the data folder. Hence, if grouppermissions are used but log_file_mode is more restrictive, a backupwith a user in the group having read access could fail even if the logdirectory is located outside of the data folder.Per discussion with the folks mentioned below, we have concluded thatcurrent_logfiles should not be treated as a log file as it only storesmetadata related to log files, and that it should use the samepermissions as all other files in the data directory. This solution hasthe merit to be simple and fixes all the interaction problems betweengroup access and log_file_mode.Author: Haribabu KommiReviewed-by: Stephen Frost, Robert Haas, Tom Lane, Michael PaquierDiscussion:https://postgr.es/m/CAJrrPGcEotF1P7AWoeQyD3Pqr-0xkQg_Herv98DjbaMj+naozw@mail.gmail.comBackpatch-through: 11, where group access has been added.
1 parente319f03 commit7d7435c

File tree

1 file changed

+19
-2
lines changed

1 file changed

+19
-2
lines changed

‎src/backend/postmaster/syslogger.c

Lines changed: 19 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
#include<sys/stat.h>
3232
#include<sys/time.h>
3333

34+
#include"common/file_perm.h"
3435
#include"lib/stringinfo.h"
3536
#include"libpq/pqsignal.h"
3637
#include"miscadmin.h"
@@ -1440,12 +1441,14 @@ set_next_rotation_time(void)
14401441
* log messages. Useful for finding the name(s) of the current log file(s)
14411442
* when there is time-based logfile rotation. Filenames are stored in a
14421443
* temporary file and which is renamed into the final destination for
1443-
* atomicity.
1444+
* atomicity. The file is opened with the same permissions as what gets
1445+
* created in the data directory and has proper buffering options.
14441446
*/
14451447
staticvoid
14461448
update_metainfo_datafile(void)
14471449
{
14481450
FILE*fh;
1451+
mode_toumask;
14491452

14501453
if (!(Log_destination&LOG_DESTINATION_STDERR)&&
14511454
!(Log_destination&LOG_DESTINATION_CSVLOG))
@@ -1458,7 +1461,21 @@ update_metainfo_datafile(void)
14581461
return;
14591462
}
14601463

1461-
if ((fh=logfile_open(LOG_METAINFO_DATAFILE_TMP,"w", true))==NULL)
1464+
/* use the same permissions as the data directory for the new file */
1465+
oumask=umask(pg_mode_mask);
1466+
fh=fopen(LOG_METAINFO_DATAFILE_TMP,"w");
1467+
umask(oumask);
1468+
1469+
if (fh)
1470+
{
1471+
setvbuf(fh,NULL,PG_IOLBF,0);
1472+
1473+
#ifdefWIN32
1474+
/* use CRLF line endings on Windows */
1475+
_setmode(_fileno(fh),_O_TEXT);
1476+
#endif
1477+
}
1478+
else
14621479
{
14631480
ereport(LOG,
14641481
(errcode_for_file_access(),

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp