CODEWITHSUNDEEP
cstringpointersc99implicit-declaration
anon_swe
anon_swe

Reputation: 9345

C compilation error for basic function

I'm trying to write my own version of strcat (I call it "append"). Here's what I have:

#include <stdio.h>

int main() {

  char *start = "start";
  char *add = "add";
  append(start, add);
  printf(start);

}

void append(char *start, char *add) {
  //get to end of start word
  char *temp = &start;
  while (*temp != '\0') {
    temp++;
  }
  *temp = *add;
  while (*temp != '\0') {
     *temp = *add; 
  }
}

When I compile, I get 3 warnings and an error:

1) warning: implicit declaration of function 'append' is invalid in C99

2) warning: format string is not a string literal (potentially insecure)

3) error: conflicting types for 'append'

I don't see how the arguments I pass into my append function within main conflict with the function definition below it.

4) warning: incompatible pointer types initializing 'char *' with an expression of type 'char **'; remove &

Why would I want to remove & here? I thought I could declare and initialize my char pointer to the proper memory address all at once.

Any help is much appreciated.

Upvotes: -2

Views: 499

Answers (4)

Sourav Ghosh
Sourav Ghosh

Reputation: 134316

1) warning: implicit declaration of function 'append' is invalid in C99

and

3) error: conflicting types for 'append'

because, you did not provide a prototype of append() before using it. You need to add a forward declaration of a function before using it. Add

void append(char *start, char *add);

before main() or put the function definition before main()

Next, in case of

 char *start = "start";
 char *add = "add";

start and add are the pointers to string literals. They are usually placed in read-only memory, means you cannot change the contents. Any attempt to do so will lead to undefined behavior.

Then, regarding

2) warning: format string is not a string literal (potentially insecure)

printf(start);

is wrong usage in this case. You need to use it like

printf("%s\n", start);

Check the man page of printf() for more details.

and finally,

4) warning: incompatible pointer types initializing 'char *' with an expression of type 'char **'; remove &

is because of

char *temp = &start;

you need to use something like

char *temp = start;   //start is a char *, no need for & here

Note: The recommended signature of main() is int main(void).

Upvotes: 3

Brian Holley
Brian Holley

Reputation: 41

C99 is pretty restrictive in how it wants you to declare things.

As Sourav said, 1 & 3 are caused by your append() function is used before it's declared in the file, which causes the compiler to generate an implicit declaration for you. Move the append() function above main() to fix that (or add a function prototype).

4 is caused by this line: char *temp = &start;

temp here is actually a char**, since you're taking the address of a char*

Upvotes: 1

Serge Ballesta
Serge Ballesta

Reputation: 148890

But this is the easy part, what the compiler could detect.

The worse part is that as you declared start as char *start = "start", it only points to an array of 6 chars (5 letters + terminating null).

So when you try to add add at its end you get Undefined Behaviour (for writing beyond and of an array) ! In that case you are writing memory where any else could be => your program is likely to break or segfault.

Upvotes: 2

Some programmer dude
Some programmer dude

Reputation: 409166

You have multiple problems with that short code. First you have

warning: implicit declaration of function 'append' is invalid in C99

The meaning of this warning is that you need to declare functions before you use them. If you don't declare a function before you use it, the compiler will have to guess its arguments and return type, and often it guesses badly.

Continuing with next warning:

warning: format string is not a string literal (potentially insecure)

This is because you provide a string variable to printf, this is, like the warning tells you, insecure. Think for example about a case where you read input from a user, and use that input as a format string to printf. What would stop the user from adding format codes in the input string? And since you don't pass arguments where would the arguments for those formats come from?

And now the error:

error: conflicting types for 'append'

This is because of the first problem, that the compiler guessed the arguments or return type of the function wrongly.

Now on to another major problem that doesn't show up as compiler errors or warnings, namely undefined behavior.

The problem is that your start and add variables points to string literals. String literal are read only (in fact, a string literal is a pointer to an array of non-modifiable characters). The first problem is that you try to modify the contents of these arrays, the second is that the arrays are only as big as needed and you are writing outside of that memory. Both these problems are causes for undefined behavior.

Upvotes: 2

Related Questions