Skip to content

Commit

Permalink
Merge pull request E3SM-Project#3002 from E3SM-Project/bartgol/eamxx/…
Browse files Browse the repository at this point in the history
…check-input-permissions
  • Loading branch information
bartgol authored Sep 18, 2024
2 parents 4e4cdf8 + ba4dea0 commit df3c8a7
Showing 1 changed file with 32 additions and 2 deletions.
34 changes: 32 additions & 2 deletions components/eamxx/cime_config/eamxx_buildnml.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
Used by buildnml. See buildnml for documetation.
"""

import os, sys, re
import os, sys, re, pwd, grp, stat, getpass
from collections import OrderedDict

import xml.etree.ElementTree as ET
Expand Down Expand Up @@ -953,7 +953,37 @@ def create_input_data_list_file(case,caseroot):
# Only add files whose full path starts with the CIME's input data location
if file_path.startswith(din_loc_root):
fd.write("scream_dl_input_{} = {}\n".format(idx, file_path))

if os.path.exists(file_path):
if os.path.isdir(file_path):
raise IsADirectoryError(f"Input file '{file_path}' is a directory, not a regular file.")
if not os.path.isfile(file_path):
raise OSError(f"Input file '{file_path}' exists but is not a regular file.")
if not os.access(file_path,os.R_OK):
try:
file_stat = os.stat(file_path)

# Get owner and group names
owner = pwd.getpwuid(file_stat.st_uid).pw_name
group = grp.getgrgid(file_stat.st_gid).gr_name

# Get file permissions
permissions = stat.filemode(file_stat.st_mode)

except Exception as e:
raise RuntimeError(f"Error retrieving file info for '{file_path}': {e}")

curr_user = getpass.getuser()
user_info = pwd.getpwnam(curr_user)
group_ids = os.getgrouplist(curr_user, user_info.pw_gid)
curr_groups = [grp.getgrgid(gid).gr_name for gid in group_ids]

raise PermissionError ("Input file exists but it is not readable for current user\n"
f" - file name: {file_path}\n"
f" - file owner: {owner}\n"
f" - file group: {group}\n"
f" - permissions: {permissions}\n"
f" - current user: {curr_user}\n"
f" - current user groups: {curr_groups}\n")

###############################################################################
def do_cime_vars_on_yaml_output_files(case, caseroot):
Expand Down

0 comments on commit df3c8a7

Please sign in to comment.