BUG: Invalid default directory and file permissions being set

[vstokesjr]

I tried to submit a bug report but all I get is You must specify a valid bug. every time I attempt to submit one.
So, I'll post it here in hopes a Sugar developer notices.

Sugar CRM Version: 5.5.0 CE
Platform: Linux/Apache (hosted)
PHP Version: 5.2.9
Note: We use SugarCRM CE on a hosted web server (i.e. not on our server or the sugar servers).

PROBLEM
Invalid file permissions are being set for directories and files being created by SugarCRM in the include/util.php (having multiple statements).

EXAMPLE / DETAILS
Additional details describing a symptom can be found in this forum post here: CACHE folder and file permission problems

SUGGESTED SOLUTION:
ALL permission settings should be set and configured in the config.php or the config_override.php. They should NOT be hard-coded deep in the code like they are now.

Example:

config_override.php:

default_permission_for_directory = 755;
default_permission_for_file = 644;
default_permission_user = '';
default_permission_group = '';


include/util.php (having multiple statements):

default_permissions' => array (
'dir_mode' => $default_permission_for_directory,
'file_mode' => $default_permission_for_file,
'chown' => $default_permission_user,
'chgrp' => $default_permission_group,

[AlexAv]

default_permission_for_directory = 755;
default_permission_for_file = 644;

It is bad permission.
It should be

default_permission_for_directory = 0755;
default_permission_for_file = 0644;

[vstokesjr]

Thank you for the corrected permissions.
IMHO the code should NEVER EVER set directory or file permissions, it should use the default user setting from the operating system. This appears to be a hack to me.

The following files have been found (so far) that set permissions:

include/utils.php
modules/UpgradeWizard/silentUpgrade.php