C / C++ / MFC

9-Dec-02 7:55

0 errors, 0 warnings.

why?! why?! why!! ! ! !

anyone want to get email and try to help me solve this?? I got an evil mysterious bug.

Ive tried 3 seperate file handling libraries , I dont even think its a file handling problem anymore. compiles and links fine, but its getting a microsoft send / dont send error dialog and crashing.

argh!!!

Maximilien9-Dec-02 8:00

Maximilien

9-Dec-02 8:00

jDawwg wrote:
0 errors, 0 warnings.

This says that there are NO compilation and link errors or warnings, it does not say that your application is without bugs and defects!

Can you tell us more ? What you are doing, where it crashes ( have you run it in a debugger ? started with the F5 key )

Max.

Rickard Andersson²⁰9-Dec-02 8:03

Rickard Andersson²⁰

9-Dec-02 8:03

jDawwg wrote:
0 errors, 0 warnings.

why?! why?! why!! ! ! !

Your code has no syntax errors.

Why it crashes is impossible to tell when you don't post the your code that crashes!

Rickard Andersson@Suza Computing
C# and C++ programmer from SWEDEN!
UIN: 50302279
E-Mail: nikado@pc.nu
Speciality: I love C#, ASP.NET and C++!

joshfl9-Dec-02 8:23

9-Dec-02 8:23

sorry

heres the code

......

	int  len;                                   // Length of returned email address.
	int  result;
	char c2[255];	                            // Storage for returned email address
	char c3[]="\0"; // Email address ignore list (pipe delimited).
	char c4[]="";            // License key (See comments above.)

<br>
	//// starting message
	AfxMessageBox("Starting The Bouncer!"); 
<br>

	/// Open the DSN Connection
	HENV hEnv = NULL; // Env Handle from SQLAllocEnv()
	HDBC hDBC = NULL; // Connection handle
	HSTMT hStmt = NULL;// Statement handle
	UCHAR szDSN[SQL_MAX_DSN_LENGTH] = "cplusmysql";// Data Source Name buffer
	UCHAR szUID[10] = "xxxx";// User ID buffer 
	UCHAR szPasswd[10] = "xxxx";// Password buffer
	UCHAR szModel[128];// Model buffer
	SDWORD cbModel;// Model buffer bytes recieved
	RETCODE retcode; // Return Code
<br>	// Allocate memory for ODBC Environment handle
	SQLAllocEnv (&hEnv);
	// Allocate memory for the connection handle
	SQLAllocConnect (hEnv, &hDBC);
<br>	// Connect to the data source using userid and password.
	retcode = SQLConnect (hDBC, szDSN, SQL_NTS, szUID, SQL_NTS, szPasswd, SQL_NTS);

<br><br>	// load api
	BOOL freeResult, runTimeLinkSuccess = FALSE; 
	HINSTANCE dllHandle = NULL;     		
	BounceCheckType BounceCheckPtr = NULL;
	dllHandle = LoadLibrary("bBounce.dll");

<br>	//Get pointer to function using GetProcAddress:
	BounceCheckPtr = (BounceCheckType)GetProcAddress(dllHandle,"BBBOUNCECHECK");

	if (retcode == SQL_SUCCESS || retcode == SQL_SUCCESS_WITH_INFO)
	{
		// Open directory for read
		CFileInfoArray fia;
		fia.AddDir(
		   m_spoolpath,                                // Directory
		   "*.*",                                      // Filemask (all files)
		   TRUE,                                       // Recurse subdirs
		   CFileInfoArray::AP_SORTBYNAME | CFileInfoArray::AP_SORTASCENDING, // Sort by name and ascending
		   FALSE                                       // Do not add array entries for directories (only for files)
		);
	
		/// here will start the directory iteration
		for (int ji=0;ji<fia.GetSize();ji++) {

		    ifstream is;
		    is.open (fia[ji].GetFilePath(), ios::binary );
		    // get length of file:
		    is.seekg (0, ios::end);
		    int length = is.tellg();
		    is.seekg (0, ios::beg);
		    // allocate memory:
		    char * c1 = new char [length+1];
		    // read data as a block:
		    is.read (c1,length);
		    is.close();
		    c1[length+1] = '\0';

			// call api function
			if(NULL != dllHandle) 
			{
				// If the function address is valid, call the bbounce api's function. 
				if (runTimeLinkSuccess = (NULL != BounceCheckPtr))
				{
					 result = BounceCheckPtr(*c1, *c2, len, *c3, *c4);
				}
			}

			///// prepare database entry
			UCHAR szSqlStr[128];
			if (result == 1) { 

				sprintf((char*)szSqlStr,"INSERT into bouncetest Values ('HB', '%s')",c2);

				SQLAllocStmt (hDBC, &hStmt); 
				// Prepare the SQL statement by assigning it to the statement handle
				SQLPrepare (hStmt, szSqlStr, sizeof (szSqlStr)); 
				// Execute the SQL statement handle
				SQLExecute (hStmt); 
				// Project only column 1 which is the models
				SQLBindCol (hStmt, 1, SQL_C_CHAR, szModel, sizeof(szModel), &cbModel);
				// Get row of data from the result set defined above in the statement
				SQLFetch (hStmt);
				// Free the allocated statement handle
				SQLFreeStmt (hStmt, SQL_DROP);
			}

			// delete memory i create on heap
			delete[] c1;

		}
		// end directory iteration

	    //Free the library:
	    freeResult = FreeLibrary(dllHandle);	

	}

	//// End the DSN Connection	
	// Disconnect from datasource
	SQLDisconnect (hDBC);
	// Free the allocated connection handle
	SQLFreeConnect (hDBC); 
	// Free the allocated ODBC environment handle
	SQLFreeEnv (hEnv);

	AfxMessageBox("Finished Sorting Bounces");

Joaquín M López Muñoz9-Dec-02 8:40

9-Dec-02 8:40

...
char * c1 = new char [length+1];
// read data as a block:
is.read (c1,length);
is.close();
c1[length+1] = '\0';
...

The last line should be

c1[length] = '\0';

This is the first thing I've noticed on a quick review, maybe some more errors are still lurking in the code.

Joaquín M López Muñoz
Telefónica, Investigación y Desarrollo

joshfl9-Dec-02 8:59

9-Dec-02 8:59

changed that to what you suggested, still crashes.

carrie9-Dec-02 9:03

carrie

9-Dec-02 9:03

If you could put a breakpoint in at the start of the function, and then Step through the code to find out where the program bombs it would be a lot of help for us to be able to find the bug.

joshfl9-Dec-02 9:43

9-Dec-02 9:43

did that,
it dies at

delete[] c1;

Smile | :)

also, i will send anyone who needs a better look the project files if you need them.

Joaquín M López Muñoz9-Dec-02 9:09

9-Dec-02 9:09

sprintf((char*)szSqlStr,"INSERT into bouncetest Values ('HB', '%s')",c2);

c2 is not initialized to anything, this is faulty. Moreover, szSqlStr is not given space enough to accomodate the maximum string possible, approx. equal to 255 plus some 50 chars.

Joaquín M López Muñoz
Telefónica, Investigación y Desarrollo

joshfl9-Dec-02 9:50

9-Dec-02 9:50

c2 is initialized. at the beginning , it is initialized to contain nothing. its just storage space for the API to write into.

at the point you are referencing to in my code, c2 contains a value that was given to it via the API function.

thanks for the suggestion , although it is not the problem. Also, szSqlStr has enough space for my needs, why do you suggest it to be bigger?

Joaquín M López Muñoz9-Dec-02 9:58

9-Dec-02 9:58

c2 is initialized. at the beginning , it is initialized to contain nothing. its just storage space for the API to write into.

By "the API" you mean that pointed to by BounceCheckPtr? What does this API do? Also, have you checked BounceCheckPtr is not null?

Also, szSqlStr has enough space for my needs, why do you suggest it to be bigger?

Well, c2 is 255 chars: without further info, seems reasonable that, under some ocassions, the strings stored in it can get near that maximum size, wich will overflow a 128 chars szSqlStr.

Joaquín M López Muñoz
Telefónica, Investigación y Desarrollo

joshfl9-Dec-02 10:33

9-Dec-02 10:33

Joaquín M López Muñoz wrote:
c2 is initialized. at the beginning , it is initialized to contain nothing. its just storage space for the API to write into.

By "the API" you mean that pointed to by BounceCheckPtr? What does this API do? Also, have you checked BounceCheckPtr is not null?

Yes, and its not null..The API determines status of a file. Im passing the contents of the file to the API via C1.
The API writes a string to c2 and returns a value as the status code of the kind of file it analyzed.

Also, szSqlStr has enough space for my needs, why do you suggest it to be bigger?
Well, c2 is 255 chars: without further info, seems reasonable that, under some ocassions, the strings stored in it can get near that maximum size, wich will overflow a 128 chars szSqlStr.

oh , ok... i see what you mean now, but i know that the c2 is only going to be between 10 - 30 charactors. really i guess i could clean it up a bit since i dont need the extra space, but i wasnt really worried about it... i was trying to figure out the mystery bug.

just to be sure tho, now that i see what you meant i tried setting szSqlStr to 400 ,still compiles 0 errors, 0 warnings, but im also still getting the mystery bug.

also since i know its the next logical concern, its also definetly not the API. The same .dll file for the API is working perfectly with a program i developed w/ VB.

Joaquín M López Muñoz9-Dec-02 10:54

9-Dec-02 10:54

Im passing the contents of the file to the API via C1.

Given the expression

BounceCheckPtr(*c1, *c2, len, *c3, *c4);

seems what you're passing is not c1, but rather the character c1[0], unless the function accepts char&s or something like that. Is that the case? If not, could it be you meant to write this?

BounceCheckPtr(c1, c2, len, c3, c4);

(Excuse me if I'm saying nonsense.)

Joaquín M López Muñoz
Telefónica, Investigación y Desarrollo