Confused by malloc and free behaviour

Question

I am confused with the usage of free() in regard to data structures in C.

If I have the following data structure:

struct Person {
    char *name;
    int age;
    float weight;
    float height;
};

I am allocating memory for the structure via malloc(), likewise: struct Person *me = malloc(sizeof(struct Person));

After I am done with using my structure (right before ending the program), I proceed to freeing the memory allocated, like this:

free(person_pointer->name);
free(person_pointer);

Freeing the memory that the name pointer points to is necessary to my knowledge, because if I only free person_pointer I only free the memory that was allocated to store the data structure and its members but not the memory that is pointed to by pointers-members of the structure.

However with my implementation valgrind seems to suggest that the first free() is invalid.

So my question boils down to: When I free the memory that a struct occupies via a pointer, should I preemptively free the memory that member pointers point to or not?

EDIT: This is my whole code:

#include <stdio.h>
#include <stdlib.h>

struct Person {
    char *name;
    int age;
    float weight;
    float height;
};

int main(int argc, char **argv)
{
    struct Person *me = malloc(sizeof(struct Person));
    me->name = "Fotis";
    me->age = 20;
    me->height = 1.75;
    me->weight = 75;

    printf("My name is %s and I am %d years old.\n", me->name, me->age);
    printf("I am %f meters tall and I weight %f kgs\n", me->height, me->weight);

    free(me->name);
    free(me);

    return 0;
}

Answer 1

me->name = "Fotis";

/* ... */

free(me->name);

The rule is:

1 malloc = 1 free

You didn't use malloc on me->name, so you don't have to free it.

BTW, me->name should be of const char * type.