Skip to content

C
core
Kaydet (Commit) 4c05911b authored tarafından Mukhiddin Yusupov's avatar Mukhiddin Yusupov Kaydeden (comit) Stephan Bergmann
Dosyalara gözat
Seçenekler

fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT 

Replaced OSL_ASSERT with SAL_WARN_IF, OSL_TRACE with SAL_INFO

Change-Id: Ia2283c09ac702558fe6ad39e963b0f401ef31de0
Signed-off-by: 's avatarStephan Bergmann <sbergman@redhat.com>
üst 197f2016
Hide whitespace changes
Inline Side-by-side
Showing with 46 additions and 350 deletions
sal/osl/unx/profile.cxx
Dosyayı görüntüle @ 4c05911b
......@@ -47,9 +47,6 @@
#define DEFAULT_PMODE (S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH | S_IWOTH)
/*#define DEBUG_OSL_PROFILE*/
/*#define TRACE_OSL_PROFILE*/
typedef time_t osl_TStamp;
typedef enum _osl_TLockMode
......@@ -153,41 +150,8 @@ static oslProfile SAL_CALL osl_psz_openProfile(const sal_Char *pszProfileName, o
osl_TProfileImpl* pProfile;
bool bRet = false;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_openProfile");
#endif
#ifdef DEBUG_OSL_PROFILE
Flags=osl_Profile_FLUSHWRITE;
OSL_TRACE("opening '%s'",pszProfileName);
if ( Flags == osl_Profile_DEFAULT )
{
OSL_TRACE("with osl_Profile_DEFAULT");
}
if ( Flags & osl_Profile_SYSTEM )
{
OSL_TRACE("with osl_Profile_SYSTEM");
}
if ( Flags & osl_Profile_READLOCK )
{
OSL_TRACE("with osl_Profile_READLOCK");
}
if ( Flags & osl_Profile_WRITELOCK )
{
OSL_TRACE("with osl_Profile_WRITELOCK");
}
if ( Flags & osl_Profile_FLUSHWRITE )
{
OSL_TRACE("with osl_Profile_FLUSHWRITE");
}
#endif
if ( ( pFile = openFileImpl(pszProfileName, Flags ) ) == NULL )
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_openProfile [not opened]");
#endif
return (NULL);
}
......@@ -212,14 +176,11 @@ static oslProfile SAL_CALL osl_psz_openProfile(const sal_Char *pszProfileName, o
pProfile->m_Stamp = OslProfile_getFileStamp(pFile);
bRet=loadProfile(pFile, pProfile);
bRet &= realpath(pszProfileName, pProfile->m_FileName) != NULL;
OSL_ASSERT(bRet);
SAL_WARN_IF(!bRet, "sal.osl", "realpath(pszProfileName, pProfile->m_FileName) != NULL ==> false");
if (pProfile->m_pFile == NULL)
closeFileImpl(pFile,pProfile->m_Flags);
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_openProfile [ok]");
#endif
return (pProfile);
}
......@@ -228,15 +189,8 @@ sal_Bool SAL_CALL osl_closeProfile(oslProfile Profile)
osl_TProfileImpl* pProfile = (osl_TProfileImpl*)Profile;
osl_TProfileImpl* pTmpProfile;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_closeProfile");
#endif
if ( Profile == 0 )
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_closeProfile [profile==0]");
#endif
return sal_False;
}
......@@ -244,11 +198,9 @@ sal_Bool SAL_CALL osl_closeProfile(oslProfile Profile)
if ( pProfile->m_bIsValid == false )
{
OSL_ASSERT(pProfile->m_bIsValid);
SAL_WARN("sal.osl", "!pProfile->m_bIsValid");
pthread_mutex_unlock(&(pProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_closeProfile [not valid]");
#endif
return sal_False;
}
......@@ -261,7 +213,7 @@ sal_Bool SAL_CALL osl_closeProfile(oslProfile Profile)
if ( pTmpProfile != 0 )
{
bool bRet = storeProfile(pTmpProfile, true);
OSL_ASSERT(bRet);
SAL_WARN_IF(!bRet, "sal.osl", "storeProfile(pTmpProfile, true) ==> false");
(void)bRet;
}
}
......@@ -273,9 +225,8 @@ sal_Bool SAL_CALL osl_closeProfile(oslProfile Profile)
if ( pTmpProfile == 0 )
{
pthread_mutex_unlock(&(pProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_closeProfile [pProfile==0]");
#endif
SAL_INFO("sal.osl", "Out osl_closeProfile [pProfile==0]");
return sal_False;
}
......@@ -326,9 +277,6 @@ sal_Bool SAL_CALL osl_closeProfile(oslProfile Profile)
free(pProfile);
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_closeProfile [ok]");
#endif
return (sal_True);
}
......@@ -338,15 +286,8 @@ sal_Bool SAL_CALL osl_flushProfile(oslProfile Profile)
osl_TFile* pFile;
bool bRet = false;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_flushProfile()");
#endif
if ( pProfile == 0 )
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_flushProfile() [pProfile == 0]");
#endif
return sal_False;
}
......@@ -354,11 +295,8 @@ sal_Bool SAL_CALL osl_flushProfile(oslProfile Profile)
if ( pProfile->m_bIsValid == false )
{
OSL_ASSERT(pProfile->m_bIsValid);
SAL_WARN_IF(!pProfile->m_bIsValid, "sal.osl", "!pProfile->m_bIsValid");
pthread_mutex_unlock(&(pProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_flushProfile [not valid]");
#endif
return sal_False;
}
......@@ -366,24 +304,16 @@ sal_Bool SAL_CALL osl_flushProfile(oslProfile Profile)
if ( !( pFile != 0 && pFile->m_Handle >= 0 ) )
{
pthread_mutex_unlock(&(pProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_flushProfile() [invalid file]");
#endif
return sal_False;
}
if ( pProfile->m_Flags & FLG_MODIFIED )
{
#ifdef DEBUG_OSL_PROFILE
OSL_TRACE("swapping to storeprofile");
#endif
bRet = storeProfile(pProfile, false);
OSL_ASSERT(bRet);
SAL_WARN_IF(!bRet, "sal.osl", "storeProfile(pProfile, false) ==> false");
}
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_flushProfile() [ok]");
#endif
pthread_mutex_unlock(&(pProfile->m_AccessLock));
return bRet;
}
......@@ -394,26 +324,19 @@ static bool writeProfileImpl(osl_TFile* pFile)
unsigned int nLen=0;
#endif
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_writeProfileImpl()");
#endif
if ( !( pFile != 0 && pFile->m_Handle >= 0 ) || ( pFile->m_pWriteBuf == 0 ) )
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_writeProfileImpl() [invalid args]");
#endif
return false;
}
#if OSL_DEBUG_LEVEL > 1
nLen=strlen(pFile->m_pWriteBuf);
OSL_ASSERT(nLen == (pFile->m_nWriteBufLen - pFile->m_nWriteBufFree));
SAL_WARN_IF(nLen != (pFile->m_nWriteBufLen - pFile->m_nWriteBufFree), "sal.osl", "nLen != (pFile->m_nWriteBufLen - pFile->m_nWriteBufFree)");
#endif
if ( !safeWrite(pFile->m_Handle, pFile->m_pWriteBuf, pFile->m_nWriteBufLen - pFile->m_nWriteBufFree) )
{
OSL_TRACE("write failed '%s'",strerror(errno));
SAL_INFO("sal.osl", "write failed " << strerror(errno));
return false;
}
......@@ -421,9 +344,7 @@ static bool writeProfileImpl(osl_TFile* pFile)
pFile->m_pWriteBuf=0;
pFile->m_nWriteBufLen=0;
pFile->m_nWriteBufFree=0;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_writeProfileImpl() [ok]");
#endif
return true;
}
......@@ -441,17 +362,10 @@ sal_Bool SAL_CALL osl_readProfileString(oslProfile Profile,
osl_TProfileImpl* pTmpProfile=0;
bool bRet = false;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_readProfileString");
#endif
pTmpProfile = (osl_TProfileImpl*) Profile;
if ( pTmpProfile == 0 )
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_readProfileString [pTmpProfile==0]");
#endif
return sal_False;
}
......@@ -460,9 +374,7 @@ sal_Bool SAL_CALL osl_readProfileString(oslProfile Profile,
if ( pTmpProfile->m_bIsValid == false )
{
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_readProfileString [not valid]");
#endif
return sal_False;
}
......@@ -471,9 +383,7 @@ sal_Bool SAL_CALL osl_readProfileString(oslProfile Profile,
if ( pProfile == NULL )
{
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_readProfileString [pProfile==0]");
#endif
return (sal_False);
}
......@@ -504,24 +414,18 @@ sal_Bool SAL_CALL osl_readProfileString(oslProfile Profile,
{ /* not implemented */ }
bRet=releaseProfile(pProfile);
OSL_ASSERT(bRet);
SAL_WARN_IF(!bRet, "sal.osl", "releaseProfile(pProfile) ==> false");
(void)bRet;
if ( pStr == 0 )
{
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_readProfileString [pStr==0]");
#endif
return sal_False;
}
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_readProfileString [ok]");
#endif
return (sal_True);
}
......@@ -533,10 +437,6 @@ sal_Bool SAL_CALL osl_readProfileBool(oslProfile Profile,
sal_Char Line[32];
Line[0] = '\0';
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_readProfileBool");
#endif
if (osl_readProfileString(Profile, pszSection, pszEntry, Line, sizeof(Line), ""))
{
if ((strcasecmp(Line, STR_INI_BOOLYES) == 0) ||
......@@ -550,10 +450,6 @@ sal_Bool SAL_CALL osl_readProfileBool(oslProfile Profile,
Default = sal_False;
}
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_readProfileBool [ok]");
#endif
return (Default);
}
......@@ -568,10 +464,6 @@ sal_uInt32 SAL_CALL osl_readProfileIdent(oslProfile Profile,
sal_Char Line[256];
Line[0] = '\0';
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_readProfileIdent");
#endif
if (osl_readProfileString(Profile, pszSection, pszEntry, Line, sizeof(Line), ""))
{
i = 0;
......@@ -586,9 +478,6 @@ sal_uInt32 SAL_CALL osl_readProfileIdent(oslProfile Profile,
}
}
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_readProfileIdent [ok]");
#endif
return (Default);
}
......@@ -606,17 +495,10 @@ sal_Bool SAL_CALL osl_writeProfileString(oslProfile Profile,
osl_TProfileImpl* pProfile = 0;
osl_TProfileImpl* pTmpProfile = 0;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_writeProfileString");
#endif
pTmpProfile = (osl_TProfileImpl*) Profile;
if ( pTmpProfile == 0 )
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_writeProfileString [pTmpProfile==0]");
#endif
return sal_False;
}
......@@ -624,11 +506,9 @@ sal_Bool SAL_CALL osl_writeProfileString(oslProfile Profile,
if ( pTmpProfile->m_bIsValid == false )
{
OSL_ASSERT(pTmpProfile->m_bIsValid);
SAL_WARN_IF(!pTmpProfile->m_bIsValid, "sal.osl", "!pTmpProfile->m_bIsValid");
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_writeProfileString [not valid]");
#endif
return sal_False;
}
......@@ -637,9 +517,7 @@ sal_Bool SAL_CALL osl_writeProfileString(oslProfile Profile,
if (pProfile == NULL)
{
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_writeProfileString [pProfile==0]");
#endif
return (sal_False);
}
......@@ -661,15 +539,11 @@ sal_Bool SAL_CALL osl_writeProfileString(oslProfile Profile,
(! addSection(pProfile, pProfile->m_NoLines - 1, &pStr[1], strlen(pszSection))))
{
bRet=releaseProfile(pProfile);
OSL_ASSERT(bRet);
SAL_WARN_IF(!bRet, "sal.osl", "releaseProfile(pProfile) ==> false");
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
free(Line);
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_writeProfileString [not added]");
#endif
return (sal_False);
}
......@@ -693,14 +567,11 @@ sal_Bool SAL_CALL osl_writeProfileString(oslProfile Profile,
(! addEntry(pProfile, pSec, i, pStr, strlen(pszEntry))))
{
bRet=releaseProfile(pProfile);
OSL_ASSERT(bRet);
SAL_WARN_IF(!bRet, "sal.osl", "releaseProfile(pProfile) ==> false");
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
free(Line);
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_writeProfileString [not inserted]");
#endif
return (sal_False);
}
......@@ -721,7 +592,7 @@ sal_Bool SAL_CALL osl_writeProfileString(oslProfile Profile,
}
bRet = releaseProfile(pProfile);
OSL_ASSERT(bRet);
SAL_WARN_IF(!bRet, "sal.osl", "releaseProfile(pProfile) ==> false");
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
if ( Line!= 0 )
......@@ -729,10 +600,6 @@ sal_Bool SAL_CALL osl_writeProfileString(oslProfile Profile,
free(Line);
}
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_writeProfileString [ok]");
#endif
return bRet;
}
......@@ -743,19 +610,11 @@ sal_Bool SAL_CALL osl_writeProfileBool(oslProfile Profile,
{
bool bRet = false;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_writeProfileBool");
#endif
if (Value)
bRet=osl_writeProfileString(Profile, pszSection, pszEntry, STR_INI_BOOLONE);
else
bRet=osl_writeProfileString(Profile, pszSection, pszEntry, STR_INI_BOOLZERO);
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_writeProfileBool [ok]");
#endif
return bRet;
}
......@@ -769,10 +628,6 @@ sal_Bool SAL_CALL osl_writeProfileIdent(oslProfile Profile,
int i, n = 0;
bool bRet = false;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_writeProfileIdent");
#endif
while (Strings[n] != NULL)
++n;
......@@ -781,9 +636,6 @@ sal_Bool SAL_CALL osl_writeProfileIdent(oslProfile Profile,
else
bRet = osl_writeProfileString(Profile, pszSection, pszEntry, Strings[i]);
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_writeProfileIdent");
#endif
return bRet;
}
......@@ -797,17 +649,10 @@ sal_Bool SAL_CALL osl_removeProfileEntry(oslProfile Profile,
osl_TProfileImpl* pTmpProfile = 0;
bool bRet = false;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_removeProfileEntry");
#endif
pTmpProfile = (osl_TProfileImpl*) Profile;
if ( pTmpProfile == 0 )
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_removeProfileEntry [pProfile==0]");
#endif
return sal_False;
}
......@@ -815,11 +660,8 @@ sal_Bool SAL_CALL osl_removeProfileEntry(oslProfile Profile,
if ( pTmpProfile->m_bIsValid == false )
{
OSL_ASSERT(pTmpProfile->m_bIsValid);
SAL_WARN_IF(!pTmpProfile->m_bIsValid, "sal.osl", "!pTmpProfile->m_bIsValid");
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_removeProfileEntry [not valid]");
#endif
return sal_False;
}
......@@ -828,9 +670,7 @@ sal_Bool SAL_CALL osl_removeProfileEntry(oslProfile Profile,
if (pProfile == NULL)
{
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_removeProfileEntry [pProfile==0]");
#endif
return (sal_False);
}
......@@ -859,13 +699,10 @@ sal_Bool SAL_CALL osl_removeProfileEntry(oslProfile Profile,
{ /* not implemented */ }
bRet = releaseProfile(pProfile);
OSL_ASSERT(bRet);
SAL_WARN_IF(!bRet, "sal.osl", "releaseProfile(pProfile) ==> false");
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_removeProfileEntry [ok]");
#endif
return bRet;
}
......@@ -881,17 +718,10 @@ sal_uInt32 SAL_CALL osl_getProfileSectionEntries(oslProfile Profile,
osl_TProfileImpl* pTmpProfile = 0;
bool bRet = false;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_getProfileSectionEntries");
#endif
pTmpProfile = (osl_TProfileImpl*) Profile;
if ( pTmpProfile == 0 )
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_getProfileSectionEntries [pTmpProfile==0]");
#endif
return sal_False;
}
......@@ -900,14 +730,10 @@ sal_uInt32 SAL_CALL osl_getProfileSectionEntries(oslProfile Profile,
if ( pTmpProfile->m_bIsValid == false )
{
OSL_ASSERT(pTmpProfile->m_bIsValid);
SAL_WARN_IF(!pTmpProfile->m_bIsValid, "sal.osl", "!pTmpProfile->m_bIsValid");
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_getProfileSectionEntries [not valid]");
#endif
return sal_False;
}
......@@ -917,10 +743,6 @@ sal_uInt32 SAL_CALL osl_getProfileSectionEntries(oslProfile Profile,
{
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_getProfileSectionEntries [pProfile=0]");
#endif
return (0);
}
......@@ -962,15 +784,11 @@ sal_uInt32 SAL_CALL osl_getProfileSectionEntries(oslProfile Profile,
}
bRet=releaseProfile(pProfile);
OSL_ASSERT(bRet);
SAL_WARN_IF(!bRet, "sal.osl", "releaseProfile(pProfile) ==> false");
(void)bRet;
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_getProfileSectionEntries [ok]");
#endif
return (n);
}
......@@ -984,17 +802,10 @@ sal_uInt32 SAL_CALL osl_getProfileSections(oslProfile Profile,
osl_TProfileImpl* pTmpProfile = 0;
bool bRet = false;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_getProfileSections");
#endif
pTmpProfile = (osl_TProfileImpl*) Profile;
if ( pTmpProfile == 0 )
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_getProfileSections [pTmpProfile==0]");
#endif
return sal_False;
}
......@@ -1002,11 +813,9 @@ sal_uInt32 SAL_CALL osl_getProfileSections(oslProfile Profile,
if ( pTmpProfile->m_bIsValid == false )
{
OSL_ASSERT(pTmpProfile->m_bIsValid);
SAL_WARN_IF(!pTmpProfile->m_bIsValid, "sal.osl", "!pTmpProfile->m_bIsValid");
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_getProfileSections [not valid]");
#endif
return sal_False;
}
......@@ -1016,9 +825,6 @@ sal_uInt32 SAL_CALL osl_getProfileSections(oslProfile Profile,
{
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_getProfileSections [pProfile==0]");
#endif
return (0);
}
......@@ -1055,15 +861,11 @@ sal_uInt32 SAL_CALL osl_getProfileSections(oslProfile Profile,
{ /* not implemented */ }
bRet=releaseProfile(pProfile);
OSL_ASSERT(bRet);
SAL_WARN_IF(!bRet, "sal.osl", "releaseProfile(pProfile) ==> false");
(void)bRet;
pthread_mutex_unlock(&(pTmpProfile->m_AccessLock));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_getProfileSections [ok]");
#endif
return (n);
}
......@@ -1086,10 +888,6 @@ static bool OslProfile_lockFile(const osl_TFile* pFile, osl_TLockMode eMode)
static bool bIsInitialized = false;
static bool bLockingDisabled;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In OslProfile_lockFile");
#endif
if ( !bIsInitialized )
{
sal_Char* pEnvValue;
......@@ -1110,17 +908,11 @@ static bool OslProfile_lockFile(const osl_TFile* pFile, osl_TLockMode eMode)
if (pFile->m_Handle < 0)
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out OslProfile_lockFile [invalid file handle]");
#endif
return false;
}
if ( bLockingDisabled )
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out OslProfile_lockFile [locking disabled]");
#endif
return true;
}
......@@ -1150,16 +942,10 @@ static bool OslProfile_lockFile(const osl_TFile* pFile, osl_TLockMode eMode)
if ( fcntl(pFile->m_Handle, F_SETLKW, &lock) == -1 && errno != ENOTSUP )
#endif
{
OSL_TRACE("fcntl returned -1 (%s)",strerror(errno));
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out OslProfile_lockFile [fcntl F_SETLKW]");
#endif
SAL_INFO("sal.osl", "fcntl returned -1 (" << strerror(errno) << ")");
return false;
}
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out OslProfile_lockFile [ok]");
#endif
return true;
}
......@@ -1171,34 +957,21 @@ static osl_TFile* openFileImpl(const sal_Char* pszFilename, oslProfileOption Pro
if ( ProfileFlags & ( osl_Profile_WRITELOCK | osl_Profile_FLUSHWRITE ) )
{
#ifdef DEBUG_OSL_PROFILE
OSL_TRACE("setting bWriteable to TRUE");
#endif
bWriteable = true;
}
if (! bWriteable)
{
#ifdef DEBUG_OSL_PROFILE
OSL_TRACE("opening '%s' read only",pszFilename);
#endif
pFile->m_Handle = open(pszFilename, O_RDONLY);
/* mfe: argghh!!! do not check if the file could be openend */
/* default mode expects it that way!!! */
}
else
{
#ifdef DEBUG_OSL_PROFILE
OSL_TRACE("opening '%s' read/write",pszFilename);
#endif
if (((pFile->m_Handle = open(pszFilename, O_RDWR | O_CREAT | O_EXCL, DEFAULT_PMODE)) < 0) &&
((pFile->m_Handle = open(pszFilename, O_RDWR)) < 0))
{
free(pFile);
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out openFileImpl [open read/write]");
#endif
return (NULL);
}
}
......@@ -1219,15 +992,9 @@ static osl_TFile* openFileImpl(const sal_Char* pszFilename, oslProfileOption Pro
if ( ProfileFlags & (osl_Profile_WRITELOCK | osl_Profile_READLOCK ) )
{
#ifdef DEBUG_OSL_PROFILE
OSL_TRACE("locking '%s' file",pszFilename);
#endif
OslProfile_lockFile(pFile, bWriteable ? write_lock : read_lock);
}
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out openFileImpl [ok]");
#endif
return (pFile);
}
......@@ -1235,15 +1002,8 @@ static osl_TStamp closeFileImpl(osl_TFile* pFile, oslProfileOption Flags)
{
osl_TStamp stamp = 0;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In closeFileImpl");
#endif
if ( pFile == 0 )
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out closeFileImpl [pFile == 0]");
#endif
return stamp;
}
......@@ -1267,42 +1027,26 @@ static osl_TStamp closeFileImpl(osl_TFile* pFile, oslProfileOption Flags)
free(pFile);
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out closeFileImpl [ok]");
#endif
return(stamp);
}
static bool OslProfile_rewindFile(osl_TFile* pFile, bool bTruncate)
{
bool bRet = true;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In osl_OslProfile_rewindFile");
#endif
if (pFile->m_Handle >= 0)
{
pFile->m_pReadPtr = pFile->m_ReadBuf + sizeof(pFile->m_ReadBuf);
#ifdef DEBUG_OSL_PROFILE
OSL_TRACE("rewinding");
#endif
bRet = (lseek(pFile->m_Handle, SEEK_SET, 0L) == 0L);
if (bTruncate)
{
#ifdef DEBUG_OSL_PROFILE
OSL_TRACE("truncating");
#endif
bRet &= (ftruncate(pFile->m_Handle, 0L) == 0);
}
}
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out osl_OslProfile_rewindFile [ok]");
#endif
return bRet;
}
......@@ -1335,7 +1079,7 @@ static sal_Char* OslProfile_getLine(osl_TFile* pFile)
if ((Max = read(pFile->m_Handle, &pFile->m_ReadBuf[Bytes], Free)) < 0)
{
OSL_TRACE("read failed '%s'",strerror(errno));
SAL_INFO("sal.osl", "read failed " << strerror(errno));
if( pLine )
rtl_freeMemory( pLine );
......@@ -1398,10 +1142,6 @@ static bool OslProfile_putLine(osl_TFile* pFile, const sal_Char *pszLine)
{
unsigned int Len = strlen(pszLine);
#ifdef DEBUG_OSL_PROFILE
int strLen=0;
#endif
if ( pFile == 0 || pFile->m_Handle < 0 )
{
return false;
......@@ -1432,9 +1172,6 @@ static bool OslProfile_putLine(osl_TFile* pFile, const sal_Char *pszLine)
}
memcpy(pFile->m_pWriteBuf + ( pFile->m_nWriteBufLen - pFile->m_nWriteBufFree ),pszLine,Len+1);
#ifdef DEBUG_OSL_PROFILE
strLen = strlen(pFile->m_pWriteBuf);
#endif
pFile->m_pWriteBuf[pFile->m_nWriteBufLen - pFile->m_nWriteBufFree + Len]='\n';
pFile->m_pWriteBuf[pFile->m_nWriteBufLen - pFile->m_nWriteBufFree + Len + 1]='\0';
......@@ -1838,7 +1575,7 @@ static bool loadProfile(osl_TFile* pFile, osl_TProfileImpl* pProfile)
{
bWasAdded = addLine( pProfile, pLine );
rtl_freeMemory( pLine );
OSL_ASSERT(bWasAdded);
SAL_WARN_IF(!bWasAdded, "sal.osl", "addLine( pProfile, pLine ) ==> false");
if ( ! bWasAdded )
return false;
}
......@@ -1864,7 +1601,7 @@ static bool loadProfile(osl_TFile* pFile, osl_TProfileImpl* pProfile)
if (! addEntry(pProfile, &pProfile->m_Sections[pProfile->m_NoSections - 1],
i, pStr, pChar - pStr))
{
OSL_ASSERT(false);
SAL_WARN("sal.osl", "Adding entry => false");
continue;
}
......@@ -1875,7 +1612,7 @@ static bool loadProfile(osl_TFile* pFile, osl_TProfileImpl* pProfile)
if (! addSection(pProfile, i, pStr + 1, pChar - pStr - 1))
{
OSL_ASSERT(false);
SAL_WARN("sal.osl", "Adding section => false");
continue;
}
......@@ -1887,10 +1624,6 @@ static bool loadProfile(osl_TFile* pFile, osl_TProfileImpl* pProfile)
static bool storeProfile(osl_TProfileImpl* pProfile, bool bCleanup)
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In storeProfile");
#endif
if (pProfile->m_Lines != NULL)
{
if (pProfile->m_Flags & FLG_MODIFIED)
......@@ -1922,9 +1655,6 @@ static bool storeProfile(osl_TProfileImpl* pProfile, bool bCleanup)
pTmpFile->m_nWriteBufLen=0;
pTmpFile->m_nWriteBufFree=0;
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out storeProfile [not flushed]");
#endif
closeFileImpl(pTmpFile,pProfile->m_Flags);
return false;
......@@ -1961,9 +1691,6 @@ static bool storeProfile(osl_TProfileImpl* pProfile, bool bCleanup)
}
}
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out storeProfile [ok]");
#endif
return true;
}
......@@ -2075,10 +1802,6 @@ static osl_TProfileImpl* acquireProfile(oslProfile Profile, bool bWriteable)
if (pProfile == NULL)
{
#ifdef DEBUG_OSL_PROFILE
OSL_TRACE("AUTOOPEN MODE");
#endif
if ( ( pProfile = (osl_TProfileImpl*) osl_openProfile(0, PFlags ) ) != NULL )
{
pProfile->m_Flags |= FLG_AUTOOPEN;
......@@ -2086,19 +1809,12 @@ static osl_TProfileImpl* acquireProfile(oslProfile Profile, bool bWriteable)
}
else
{
#ifdef DEBUG_OSL_PROFILE
OSL_TRACE("try to acquire");
#endif
if (! (pProfile->m_Flags & osl_Profile_SYSTEM))
{
if (! (pProfile->m_Flags & (osl_Profile_READLOCK | osl_Profile_WRITELOCK | osl_Profile_FLUSHWRITE )))
{
osl_TStamp Stamp;
#ifdef DEBUG_OSL_PROFILE
OSL_TRACE("Profile acquire DEFAULT MODE");
#endif
if (! (pProfile->m_pFile = openFileImpl(pProfile->m_FileName, pProfile->m_Flags | PFlags )))
return NULL;
......@@ -2111,15 +1827,12 @@ static osl_TProfileImpl* acquireProfile(oslProfile Profile, bool bWriteable)
pProfile->m_Stamp = Stamp;
bRet=loadProfile(pProfile->m_pFile, pProfile);
OSL_ASSERT(bRet);
SAL_WARN_IF(!bRet, "sal.osl", "loadProfile(pProfile->m_pFile, pProfile) ==> false");
(void)bRet;
}
}
else
{
#ifdef DEBUG_OSL_PROFILE
OSL_TRACE("Profile acquire READ/WRITELOCK MODE");
#endif
/* A readlock file could not be written */
if ((pProfile->m_Flags & osl_Profile_READLOCK) && bWriteable)
{
......@@ -2134,36 +1847,23 @@ static osl_TProfileImpl* acquireProfile(oslProfile Profile, bool bWriteable)
static bool releaseProfile(osl_TProfileImpl* pProfile)
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("In releaseProfile");
#endif
if ( pProfile == 0 )
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out releaseProfile [profile==0]");
#endif
return false;
}
if (pProfile->m_Flags & FLG_AUTOOPEN)
{
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out releaseProfile [AUTOOPEN]");
#endif
return (osl_closeProfile((oslProfile)pProfile));
}
else
{
#ifdef DEBUG_OSL_PROFILE
OSL_TRACE("DEFAULT MODE");
#endif
if (! (pProfile->m_Flags & (osl_Profile_READLOCK | osl_Profile_WRITELOCK | osl_Profile_FLUSHWRITE )))
{
if (pProfile->m_Flags & FLG_MODIFIED)
{
bool bRet = storeProfile(pProfile, false);
OSL_ASSERT(bRet);
SAL_WARN_IF(!bRet, "sal.osl", "storeProfile(pProfile, false) ==> false");
(void)bRet;
}
......@@ -2172,9 +1872,6 @@ static bool releaseProfile(osl_TProfileImpl* pProfile)
}
}
#ifdef TRACE_OSL_PROFILE
OSL_TRACE("Out releaseProfile [ok]");
#endif
return true;
}
......
sal/osl/unx/security.cxx
Dosyayı görüntüle @ 4c05911b
......@@ -61,9 +61,8 @@ static bool sysconf_SC_GETPW_R_SIZE_MAX(std::size_t * value) {
way and always set EINVAL, so be resilient here: */
return false;
} else {
OSL_ASSERT(
m >= 0
&& (unsigned long) m < std::numeric_limits<std::size_t>::max());
SAL_WARN_IF( m < 0 || (unsigned long) m >= std::numeric_limits<std::size_t>::max(), "sal.osl",
"m < 0 || (unsigned long) m >= std::numeric_limits<std::size_t>::max()");
*value = (std::size_t) m;
return true;
}
......@@ -218,7 +217,7 @@ sal_Bool SAL_CALL osl_getUserIdent(oslSecurity Security, rtl_uString **ustrIdent
bRet = osl_psz_getUserIdent(Security,pszIdent,sizeof(pszIdent));
rtl_string2UString( ustrIdent, pszIdent, rtl_str_getLength( pszIdent ), osl_getThreadTextEncoding(), OUSTRING_TO_OSTRING_CVTFLAGS );
OSL_ASSERT(*ustrIdent != NULL);
SAL_WARN_IF(*ustrIdent == NULL, "sal.osl", "*ustrIdent == NULL");
return bRet;
}
......@@ -252,7 +251,7 @@ sal_Bool SAL_CALL osl_getUserName(oslSecurity Security, rtl_uString **ustrName)
bRet = osl_psz_getUserName(Security,pszName,sizeof(pszName));
rtl_string2UString( ustrName, pszName, rtl_str_getLength( pszName ), osl_getThreadTextEncoding(), OUSTRING_TO_OSTRING_CVTFLAGS );
OSL_ASSERT(*ustrName != NULL);
SAL_WARN_IF(*ustrName == NULL, "sal.osl", "ustrName == NULL");
return bRet;
}
......@@ -281,7 +280,7 @@ sal_Bool SAL_CALL osl_getHomeDir(oslSecurity Security, rtl_uString **pustrDirect
if ( bRet )
{
rtl_string2UString( pustrDirectory, pszDirectory, rtl_str_getLength( pszDirectory ), osl_getThreadTextEncoding(), OUSTRING_TO_OSTRING_CVTFLAGS );
OSL_ASSERT(*pustrDirectory != NULL);
SAL_WARN_IF(*pustrDirectory == NULL, "sal.osl", "*pustrDirectory == NULL");
osl_getFileURLFromSystemPath( *pustrDirectory, pustrDirectory );
}
......@@ -376,7 +375,7 @@ sal_Bool SAL_CALL osl_getConfigDir(oslSecurity Security, rtl_uString **pustrDire
if ( bRet )
{
rtl_string2UString( pustrDirectory, pszDirectory, rtl_str_getLength( pszDirectory ), osl_getThreadTextEncoding(), OUSTRING_TO_OSTRING_CVTFLAGS );
OSL_ASSERT(*pustrDirectory != NULL);
SAL_WARN_IF(*pustrDirectory == NULL, "sal.osl", "*pustrDirectory == NULL");
osl_getFileURLFromSystemPath( *pustrDirectory, pustrDirectory );
}
......@@ -422,19 +421,19 @@ static bool SAL_CALL osl_psz_getConfigDir(oslSecurity Security, sal_Char* pszDir
struct stat st;
if (stat(pszDirectory, &st) != 0)
{
OSL_TRACE("Could not stat $HOME/.config");
SAL_INFO("sal.osl","Could not stat $HOME/.config");
dirOK = false;
}
else
{
if (!S_ISDIR(st.st_mode))
{
OSL_TRACE("$HOME/.config is not a directory");
SAL_INFO("sal.osl", "$HOME/.config is not a directory");
dirOK = false;
}
if (!(st.st_mode & S_IRUSR && st.st_mode & S_IWUSR && st.st_mode & S_IXUSR))
{
OSL_TRACE("$HOME/.config has bad permissions");
SAL_INFO("sal.osl", "$HOME/.config has bad permissions");
dirOK = false;
}
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment