Strange chars
-
There might be some other way that the library determines the length of the string. Is there any documentation? Otherwise you can try adding this line of code before printing the string:
dir_struct->directory[DIR_NAME_LEN - 1] = 0; <<======================This line of code
TRACE("\n=>%s\n", dir_struct->directory);
The difficult we do right away... ...the impossible takes slightly longer.
-
I am trying to convert a C Linux library to Windows. And I've met something weird. I have a variable:
struct dir_struct
{
void* display;
char directory[DIR_NAME_LEN];
...
....but when I listed this variable on my debugger:
TRACE("\n=>%s\n", dir_struct->directory);
Result:
=>/Wise Registry Cleaner/Oe©*Bº cE«Mòצ£š{@Q—Y|†Š˜ b!//„]Û¡¼à k3©‘ø¼/´4þ“Ðõ÷düJ¤ $èû8©KªõÝ5ø³/ ò|”Áæpç5V—{?///1Ðq$\ß=*C±ì///¶//ˆòíÔ¤ö]¨m´'ȈÊÉØ£á€MÃ=¿U“œ*CÜFeR//ˆ¹5»mŠ€€†Â#øD/×c|ÔÂq,’X`t¸I//TÁ ½w¢u,¸ ß÷ìÔ¶‚Ô„ ;% ÀÔI¯¤w¡"lï ÌC8ñÅÿ/ùê‡ ŒmŸ‹àúÚÃÓÍ3ãµ+TG¯@qܾA¬t´+vÛutô|‰Ä~:žÞHÏê6'9B~o½/þŒ#ƒ-©¶Ô"÷³'£//"Ù³{R”DLUDz] œ 1 ½&{ÙòMÇ–_o™ºEí /ôež…°Û¬”¢ ¿‚yÚÊ£Cýñ:(×N±/’z¹ >eV®//§þ,÷ ‹ËÀòTÀÃS ////////»& Þ¦)ûªÝvlleŽ/xØB¡RÚ_Ò^6y¿w`žs¤‘øÂÂO{;ªqûN/0€È#@–ñyøq½nç// Ëä8íþ»¡R5½ÐºˆÍ////////////Why I have this garbage ? Because my folder is Wise Registry Cleaner only ... If I would understand it, maybe I can get rid of it ... -
What is the complete definition of the
dir_structstructure? In Linux the information provided by the readdir(3) - Linux manual page[^] system call provides a properly null terminated string. -
How do you obtain the file name from Windows? Which API function call are you using? If the documentation for that API function call specifies that the name is null terminated, then you should not be getting the garbage. But some old file systems set off a fixed length buffer for the file name - even in Unix! The original Unix directory was 16 bytes per entry: two bytes inode, 14 chars filename. The orginal FAT had room for 8 char file name, 3 char extension. If the field was filled with a maximum length file name (in either Unix or FAT), there was no room for a terminating nul. If you reach the end of the field before finding a null, it is nevertheless the end of the field! Remember that nul terminated strings is an idea conceived for the C language. It directoy conflicts with the ISO character set standard, whether ISO 646, 8859 or other variants. Other languages have used other solutions, e.g. in Pascal, a string is (at least in most implementations) represented by a length field associated with the string buffer (so the ISO NUL character can be used in the ISO standard way!). Many other languages to the same. DOS was developed before C and Unix. Windows was developed before Unix had become very important, at least in the market segments Windows was aimed at. So you should not take for granted that all sorts of C / Unix conventions have been adopeted in every other segment.
-
#define DIR_NAME_LEN 2048
struct dir_struct
{
void* display;
char directory[DIR_NAME_LEN];
...
....I have followed the way of how this variable is handled:
dir_struct_t dir_struct;
TRACE(\_T("Directory - %s\\n"), dir\_struct.directory); -> \_CrtDbgReport: String too long or IO Error strncpy(dir\_struct->directory, "/", sizeof(dir\_struct->directory)); TRACE(\_T("Directory - %s\\n"), dir\_struct->directory); -> / strcat(dir\_struct->directory, current\_file->name); TRACE(\_T("Directory - %s\\n"), dir\_struct->directory); -> \_CrtDbgReport: String too long or IO ErrorAnd if I look into VS debugger when I got "_CrtDbgReport: String too long or IO Error" from TRACE macro, I see the following result: see image.[^] and Untitled2 — Postimage.org[^] The line
strcat(dir_struct->directory, current_file->name);is not quite correct ? I mean, correct for Windows, even if is correct for Linux ... -
#define DIR_NAME_LEN 2048
struct dir_struct
{
void* display;
char directory[DIR_NAME_LEN];
...
....I have followed the way of how this variable is handled:
dir_struct_t dir_struct;
TRACE(\_T("Directory - %s\\n"), dir\_struct.directory); -> \_CrtDbgReport: String too long or IO Error strncpy(dir\_struct->directory, "/", sizeof(dir\_struct->directory)); TRACE(\_T("Directory - %s\\n"), dir\_struct->directory); -> / strcat(dir\_struct->directory, current\_file->name); TRACE(\_T("Directory - %s\\n"), dir\_struct->directory); -> \_CrtDbgReport: String too long or IO ErrorAnd if I look into VS debugger when I got "_CrtDbgReport: String too long or IO Error" from TRACE macro, I see the following result: see image.[^] and Untitled2 — Postimage.org[^] The line
strcat(dir_struct->directory, current_file->name);is not quite correct ? I mean, correct for Windows, even if is correct for Linux ...You are making a wrong assumption. The
sizeof(dir_struct->directory)operator will return2048. But what if the string in that table is less than that? You need an actual count of the number of valid characters in the directory field (assuming that it is correctly null terminated). You are also mixing the dot (.) and arrow (->) operators on your struct reference. But all this is largely irrelevant. You need to go back a stage to the code that fills the directory structure in the first place, as that is where the problem is most likely to be found. -
#define DIR_NAME_LEN 2048
struct dir_struct
{
void* display;
char directory[DIR_NAME_LEN];
...
....I have followed the way of how this variable is handled:
dir_struct_t dir_struct;
TRACE(\_T("Directory - %s\\n"), dir\_struct.directory); -> \_CrtDbgReport: String too long or IO Error strncpy(dir\_struct->directory, "/", sizeof(dir\_struct->directory)); TRACE(\_T("Directory - %s\\n"), dir\_struct->directory); -> / strcat(dir\_struct->directory, current\_file->name); TRACE(\_T("Directory - %s\\n"), dir\_struct->directory); -> \_CrtDbgReport: String too long or IO ErrorAnd if I look into VS debugger when I got "_CrtDbgReport: String too long or IO Error" from TRACE macro, I see the following result: see image.[^] and Untitled2 — Postimage.org[^] The line
strcat(dir_struct->directory, current_file->name);is not quite correct ? I mean, correct for Windows, even if is correct for Linux ...You have actually identified your problem, you just haven't realized it.
strncpy(dir_strut->directory, "/", sizeof(dir_struct_directory)); // here name is good
strcat(dir_struct->directory, current_file->name); // here name is badIt would seem that current_file->name is not a null terminated C string. You have not given us the details of
current_file, or how it is instantiated, so we can't further diagnose the problem at the moment. Perhapscurrent_filehas a member that tells you how long thenamemember is, in which case you should use that andstrncatto append the file name to the directory name.Keep Calm and Carry On
-
You are making a wrong assumption. The
sizeof(dir_struct->directory)operator will return2048. But what if the string in that table is less than that? You need an actual count of the number of valid characters in the directory field (assuming that it is correctly null terminated). You are also mixing the dot (.) and arrow (->) operators on your struct reference. But all this is largely irrelevant. You need to go back a stage to the code that fills the directory structure in the first place, as that is where the problem is most likely to be found. -
"You are also mixing the dot (.) and arrow (->) operators on your struct reference." Yes, you are right, I taken those lines of codes from different function (in one of them was reference, in other was pointer), that why those mixing are there.
Well you should not be doing so, since the reference in both cases is a structure, not a pointer. The dot operator is the correct one. But more importantly you have still not shown or explained when and where this structure gets initialised, and the directory name copied in. Until you give us that information anything we pass on is pure guesswork.
-
You have actually identified your problem, you just haven't realized it.
strncpy(dir_strut->directory, "/", sizeof(dir_struct_directory)); // here name is good
strcat(dir_struct->directory, current_file->name); // here name is badIt would seem that current_file->name is not a null terminated C string. You have not given us the details of
current_file, or how it is instantiated, so we can't further diagnose the problem at the moment. Perhapscurrent_filehas a member that tells you how long thenamemember is, in which case you should use that andstrncatto append the file name to the directory name.Keep Calm and Carry On
Yes, indeed, from this point
strcat(dir_struct->current_directory, current_file->name);, mydir_struct.directoryseem to go crazy. current_file->name is coming from:const file_info* current_file = AnotherFunction();
and file_info is a struct defined like this:
typedef struct
{
char* name;
.....
unsigned int status;
}file_info;Here is the steps:
strncpy(dir_struct->directory, "/", sizeof(dir_struct->directory));
TRACE(_T("Directory - %s\n"), dir_struct->directory); -> Directory - /
const file_info* current_file = AnotherFunction();
TRACE("\n===>>>%s\n", current_file->name); ->===>>>Unbroken
TRACE(_T("Directory - %s\n"), dir_struct->directory); -> _CrtDbgReport: String too long or IO Error
-
Yes, indeed, from this point
strcat(dir_struct->current_directory, current_file->name);, mydir_struct.directoryseem to go crazy. current_file->name is coming from:const file_info* current_file = AnotherFunction();
and file_info is a struct defined like this:
typedef struct
{
char* name;
.....
unsigned int status;
}file_info;Here is the steps:
strncpy(dir_struct->directory, "/", sizeof(dir_struct->directory));
TRACE(_T("Directory - %s\n"), dir_struct->directory); -> Directory - /
const file_info* current_file = AnotherFunction();
TRACE("\n===>>>%s\n", current_file->name); ->===>>>Unbroken
TRACE(_T("Directory - %s\n"), dir_struct->directory); -> _CrtDbgReport: String too long or IO Error
The AnotherFunction itself or the way you are using it seems to be a reason of your problem... :|
-
Yes, indeed, from this point
strcat(dir_struct->current_directory, current_file->name);, mydir_struct.directoryseem to go crazy. current_file->name is coming from:const file_info* current_file = AnotherFunction();
and file_info is a struct defined like this:
typedef struct
{
char* name;
.....
unsigned int status;
}file_info;Here is the steps:
strncpy(dir_struct->directory, "/", sizeof(dir_struct->directory));
TRACE(_T("Directory - %s\n"), dir_struct->directory); -> Directory - /
const file_info* current_file = AnotherFunction();
TRACE("\n===>>>%s\n", current_file->name); ->===>>>Unbroken
TRACE(_T("Directory - %s\n"), dir_struct->directory); -> _CrtDbgReport: String too long or IO Error
I have just tried to reproduce what you have done and the code works fine. I can only conclude that the code you are showing does not match what you are actually running. But since you have only shown small pieces and not the complete method where this occurs, it is difficult to say more.
-
I have just tried to reproduce what you have done and the code works fine. I can only conclude that the code you are showing does not match what you are actually running. But since you have only shown small pieces and not the complete method where this occurs, it is difficult to say more.
-
I really, really would show the entire code because I am struggling of it, but unfortunately I cannot due to confidentiality issue. I'll try to write more code here.
What you need to display is the assignment to dir_struct.directory - nothing more. It may be as simple as a single statement (/line). I guess that confidentiality issues won't preclude that. How did you get hold of that value? Which system call or library function provided it? If you say "None, we built it from pieces ourselves, and that code is confidential", then you have got the answer: In that string building code you forgot to add the terminating nul. Otherwise: Tell us who provided the directory string, in which way.
-
I really, really would show the entire code because I am struggling of it, but unfortunately I cannot due to confidentiality issue. I'll try to write more code here.
-
What you need to display is the assignment to dir_struct.directory - nothing more. It may be as simple as a single statement (/line). I guess that confidentiality issues won't preclude that. How did you get hold of that value? Which system call or library function provided it? If you say "None, we built it from pieces ourselves, and that code is confidential", then you have got the answer: In that string building code you forgot to add the terminating nul. Otherwise: Tell us who provided the directory string, in which way.
No, this code is not written by us. I'll try to write here as much as I can. P.S. The buggy code is written in C for Linux, and I don't know C (I guess this is visible :D) P.S. I don't pretend to know C++ either :D ... only that I fight in this domain ....
-
I am sure that you can modify any confidential pieces so they do not reveal anything that they should not.
-
info* current = list_entry(walker, const info, list);
TRACE(_T("current->name %s\n"), current->name); -> _CrtDbgReport: String too long or IO ErrorWhy I got _CrtDbgReport: String too long or IO Error I wrote in my previous posts ...
I have managed to create a sample of all that code, although I find the
#define list_entry_constsomewhat difficult to deconstruct. However, with a valid name in thefile_infoentry it works fine. I can only assume that somewhere in the actual code that you are running there is some corruption of your data, or a pointer is not set up correctly. [edit] I also could not find the TRACE macro in Windows. [/edit] -
I have managed to create a sample of all that code, although I find the
#define list_entry_constsomewhat difficult to deconstruct. However, with a valid name in thefile_infoentry it works fine. I can only assume that somewhere in the actual code that you are running there is some corruption of your data, or a pointer is not set up correctly. [edit] I also could not find the TRACE macro in Windows. [/edit]Richard MacCutchan wrote:
I also could not find the TRACE macro in Windows.
Isn't that an MFC thing?
"One man's wage rise is another man's price increase." - Harold Wilson
"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons
"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles
