My dynamically allocated 2d array print random gibberish with no meaning - c

Question

This code can get the first row to print, but for some reason will not get the first column to print. If this clarifies the question a bit the x values print but the y value barely does, I get some weird characters instead.
char** GenerateLand(int width, int height) {
char** GenLand;
int i;
GenLand = malloc(width * sizeof(char*));
for(i = 0; i < width; i++) {
GenLand[i] = malloc(sizeof(char) * height);
}
char RandChar = PickLandType();
int x, y;
for(x = 0; x < width; x++) {
if(x == 0) {
GenLand[x][0] = RandChar;
} else {
char RandChar1 = GenerateNeighbor(RandChar);
GenLand[x-1][0] = GenerateNeighbor(RandChar1);
for( y = 0; y < height ; y++){
if(y == 0){
GenLand[x][y] = RandChar;
} else {
char RandChar2 = GenerateNeighbor(RandChar);
GenLand[0][y-1] = GenerateNeighbor(RandChar2);
}
}
}
return GenLand;
for(i = 0; i < width; i++) {
free(GenLand[i]);
}
free(GenLand);
}
char GenerateNeighbor(char item)
{
switch(item)
{
case 'R':
return NewFromRuralLand();
break;
case 'F':
return NewFromForest();
break;
case 'T':
return NewFromTown();
break;
case 'W':
return NewFromWater();
break;
case 'C':
return NewFromCity();
break;
case 'M':
return NewFromMountain();
break;
default:
printf("Error!");
break;
}
}
char PickLandType()
{
int typeOfLand = rand()%5;
if(typeOfLand == 0){
return 'R';
} else if(typeOfLand == 1){
return 'F';
} else if(typeOfLand == 2){
return 'T';
} else if(typeOfLand == 3){
return 'W';
} else if(typeOfLand == 4){
return 'C';
} else if(typeOfLand == 5){
return 'M';
}
}
void PrintLand(char** Land, int width, int height) {
int x, y;
for( x = 0; x < width; x++){
for( y = 1; y < height ; y++){
printf("%c", Land[x][y]);
}
}
}
https://i.stack.imgur.com/t6Fec.png ^ Picture of error, was hoping someone could tell me why I keep getting these cryptic unreadable values. Sorry I don't have enough reputation points to directly embed the picture. I checked my NewFrom() functions and they all work so I do not believe they are the issue. The output is supposed to be organized by row and column:

Answer 1

I don't have enough reputation to comment, hence I have to write this as an answer.
Your freeing code at the end of GenerateLand will never run, there is an unconditional return statement prior to it. (NB: You shouldn't be freeing it regardless as you want to return GenLand, just remove the freeing code)
Without giving us the PickLandType and GenerateNeighbor functions, we can't tell you why your code returns a garbled mess of letters. Please edit and attach these functions.
This is the exact same question you asked before, except now you've attached different output - have you changed the code inbetween? If so, please attach the updated code.
This is bad code, the first iteration will access GenLand[0][-1]:
for(y = 0; y < height; y++) {
GenLand[0][y-1] = GenerateNeighbor(RandChar1);
}
Please try and keep your code nicely formatted to that it's easier for everyone to read.
If you want an answer that can actually help, give the GenerateNeighbor and PickLandType functions, and also give us an expected output.
Edit:
As Paul Ogilvie pointed out, the code in point 4 will also only ever generate the 0th column (the left most column). You probably meant for this to be GenLand[x][y] (but once again, see point 4)