Compiling error

Dear All,

I  compiled a project under Unix,  I have got the following errors:

------------------------------------------
 CC -I. -DUNIX -c XPCFileStat.C -g -o XPCFileStat.o
"XPCFileStat.C", line 11: Warning: String literal converted to char* in formal argument sMsg in call to XPCException::XPCException(char*).
"XPCFileStat.C", line 20: Warning: String literal converted to char* in formal argument sMsg in call to XPCException::XPCException(char*).
"XPCFileStat.C", line 28: Warning: String literal converted to char* in formal argument sMsg in call to XPCException::XPCException(char*).
"XPCFileStat.C", line 36: Warning: String literal converted to char* in formal argument sMsg in call to XPCException::XPCException(char*).
"XPCFileStat.C", line 48: Warning: String literal converted to char* in formal argument sMsg in call to XPCException::XPCException(char*).
"XPCFileStat.C", line 59: Warning: String literal converted to char* in formal argument sMsg in call to XPCException::XPCException(char*).
"XPCFileStat.C", line 70: Warning: String literal converted to char* in formal argument sMsg in call to XPCException::XPCException(char*).
"XPCFileStat.C", line 77: Error: The "&" operator can only be applied to a variable or other l-value.
"XPCFileStat.C", line 221: Warning: String literal converted to char* in formal argument sMsg in call to XPCException::XPCException(char*).
"XPCFileStat.C", line 228: Error: The "&" operator can only be applied to a variable or other l-value.
2 Error(s) and 8 Warning(s) detected.
*** Error code 2
make: Fatal error: Command failed for target `XPCFileStat.o'
------------------------------------------------------

Here is the main part of the program:

#include <XPCFileStat.h>
#include <iostream.h>

XPCFileStat::XPCFileStat()
{
    long lMaxpath;

    // Determine the maximum size of a pathname
    if ((lMaxpath = pathconf("/", _PC_PATH_MAX)) == -1)
    {
        XPCException newExcept("Could not determine maximum pathname length"); // warning line 11
        throw newExcept;
        return;
    }

    // Allocate memory for the pathname
    cFileName = new char[lMaxpath + 1];
    if (!cFileName)
    {
        XPCException newExcept("Could not allocate memory for cFileName"); // warning line 20
        throw newExcept;
        return;
    }

    // Store the current working directory
    if (getcwd(cFileName, lMaxpath) == NULL)
    {
        XPCException newExcept("Could not get current working directory"); // warning line 28
        throw newExcept;
        return;
    }

    // Retrieve the file's statistics
    if (lstat(cFileName, &sStatBuf) == -1)
    {
        XPCException newExcept("Could not obtain statics on directory."); // warning line 36
        throw newExcept;
        return;
    }
}

XPCFileStat::XPCFileStat(char *_psFileName)
{
    // Allocate memory to store the pathname
    cFileName = new char[strlen(_psFileName)+1];
    if (!cFileName)
    {
        XPCException newExcept("Could not allocate memory for cFileName"); // warming line 48
        throw newExcept;
        return;
    }

    // Copy the pathname to the private data member
    strcpy(cFileName, _psFileName);

    // Retrieve the file's statistics
    if (lstat(cFileName, &sStatBuf) == -1)
    {
        XPCException newExcept("Could not obtain statics on directory."); // warning line 59
        throw newExcept;
        return;
    }    
}

XPCFileStat::XPCFileStat(const XPCFileStat &_oldClass)
{
    cFileName = new char[sizeof(_oldClass.cFileName)+1];
    if (!cFileName)
    {
        XPCException newExcept("Could not allocate memory for cFileName");// warning line 70
        throw newExcept;
        return;
    }

    strcpy(cFileName, _oldClass.sGetFileName());

    memcpy((void *)&sStatBuf, (void *)&_oldClass.getStatBuf(), sizeof(struct stat)); //here is first error
}

enum eDirectoryTypes XPCFileStat::iGetFileType()
{
    // Extract the file type bits from st_mode and match them up with
    // the file type constants

    switch(sStatBuf.st_mode & S_IFMT)
    {
       ..............................
    }
}

enum ePermissions XPCFileStat::iGetOwnerPermissions()
{
    // Extract the owner permission bits and return the appropriate
    // permission value

    switch(sStatBuf.st_mode & S_IRWXU)
    {
        .......................................................
    }
}

enum ePermissions XPCFileStat::iGetGroupPermissions()
{
    // Extract the group permission bits and return the appropriate
        // permission value

    switch(sStatBuf.st_mode & S_IRWXG)
    {
                ..................................................
    }
}

enum ePermissions XPCFileStat::iGetOtherPermissions()
{
    // Extract the "other users" permission bits and return the appropriate
    // permission value

    switch(sStatBuf.st_mode & S_IRWXO)
    {
                ......................................
    }
}

XPCFileStat &XPCFileStat::operator=(const XPCFileStat &_oldClass)
{
    if (this == &_oldClass)
        return *this;

    if (sizeof(cFileName) < sizeof(_oldClass.sGetFileName()))
    {
        delete [] cFileName;
        cFileName = new char[sizeof(cFileName)];
        if (!cFileName)
        {
                    XPCException newExcept("Could not allocate memory for cFileName"); // warning line 221
                    throw newExcept;
                    return *this;
        }    
    }
   
    memcpy((void *)cFileName, (void *)_oldClass.sGetFileName(), sizeof(_oldClass.sGetFileName()));
    memcpy((void *)&sStatBuf, (void *)&_oldClass.getStatBuf(), sizeof(struct stat)); // here is the 2 error
}

=======================================================================

I would be glag if someone would help me to solve at least those two errors. Your help will be appreciated!

Thank you in advance for your reply.
Regards,
Azzedine
azzedineAsked:
Who is Participating?

[Product update] Infrastructure Analysis Tool is now available with Business Accounts.Learn More

x
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

grg99Commented:
The error on lines 77 and 228 is because you're trying to get the address of a method, you shouldnt need the & at all, if as I suspect gewtstatbuf returns a pointer.

The other messages are warnings, but why passing a literal to a char * is a warning is beyond me.
Maybe you could post the declarations for that newExcept method?







0

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
KashraCommented:
The warnings come from trying to pass a 'string literal' (which means something you type between quotation marks in your code) to a function that asks for a char *.

This is unsafe because string literals are not supposed to be modified, they are const objects. Therefore they should only be passed to functions that ask for a const char * as their parameter. I'm surprised this is just a warning and not an error message. Either change the function to accept a const char*, or construct the C-style strings ahead of time before passing them to the function.

For your errors:

memcpy((void *)&sStatBuf, (void *)&_oldClass.getStatBuf(), sizeof(struct stat));

If getStatBuf returns a pointer, as noted by grg99, you shouldn't need to use &. But if it returns an object by reference, then try:
memcpy((void *)&sStatBuf, (void *)&(_oldClass.getStatBuf()), sizeof(struct stat));

And likewise with the second error. Good luck!
0
tinchosCommented:
No comment has been added lately, so it's time to clean up this TA.
I will leave the following recommendation for this question in the Cleanup topic area:

Split: grg99 {http:#9725113} & Kashra {http:#9726778}

Please leave any comments here within the next seven days.
PLEASE DO NOT ACCEPT THIS COMMENT AS AN ANSWER!

Tinchos
EE Cleanup Volunteer
0
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
C++

From novice to tech pro — start learning today.