Consistently Getting Null Value in C String using getcwd

Question

I am trying to make a simple program that just writes your working directory to a file, and I cannot, for the life of me, figure out what I am doing wrong. No matter what I do, my buffer is storing null after my call to getcwd(). I suspect it may have to do with permissions, but allegedly, linux now did some wizardry to ensure that getcwd almost never has access problems (keyword, "almost"). Can anyone test it on their machines? Or is there an obvious bug I am missing?

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

int main(int argc, char *argv[])
{
        printf("Error is with fopen if stops here\n");
        FILE* out_file = fopen("dir_loc.sh","w+");
        char* loc = malloc(sizeof(char)*10000);
        size_t size = sizeof(loc);
        printf("Error is with cwd if stops here\n");
        loc = getcwd(loc,size);
        printf("%s",loc);
        fprintf(out_file,"cd %s",loc);
        printf("Error is with fclose if stops here\n");
        free(loc);
        fclose(out_file);
        return 0;
}

compiled with gcc main.c (the file is named "main.c")

EDIT: As was mentioned by different posters, sizeof(loc) was taking the size of a char pointer, and not the size of the amount of space allocated to that pointer. Changed it to malloc(sizeof(char)*1000) and it all works gravy.

Answer 1

Your problem is here:

size_t size = sizeof(loc);

You're getting the size of a char pointer, not the allocated memory for your char.

Change it to:

size_t size = sizeof(char) * 10000;

or even to

size_t size = 10000;

since sizeof(char) is guaranteed to be 1.

And since you're using size in your subsequent call to getcwd, you're obviously gonna have too little space to store most paths, so your result is unsurprising

If you don't want to go about changing multiple different numbers in the code every time you make a change, you can use #DEFINE text replacement to solve that.

Like this:

#include <string.h>
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#define LOC_ARRAY_SIZE 10000 // Here you define the array size

int main(int argc, char *argv[])
{
        printf("Error is with fopen if stops here\n");
        FILE* out_file = fopen("dir_loc.sh","w+");
        char* loc = malloc(sizeof(char)*LOC_ARRAY_SIZE); // sizeof(char) could be omitted
        size_t size = sizeof(char)*LOC_ARRAY_SIZE;
        printf("Error is with cwd if stops here\n");
        loc = getcwd(loc,size);
        printf("%s",loc);
        fprintf(out_file,"cd %s",loc);
        printf("Error is with fclose if stops here\n");
        free(loc);
        fclose(out_file);
        return 0;
}