Odd declaration

Hello everyone, I've been running into some problems writing code for my boss and I was wondering if you could help me. You see, I have a function that is called after clicking a button on a GUI. Inside this function I make the declaration:

Code:

---

SuspendCount=PortThread[PortNumber-1]->ResumeThread();

---

Where PortThread is a CWinThread* and SuspendCount and the return on ResumeThread() are DWORDs. The odd occurance arises when debugging the same declaration in a previous version of the code that I am trying to re-create. When doing so, stepping into (or over, for that matter) that line of code, it initiates this line of code:

Code:

---

DWORD aword = ::WaitForMultipleObjects(2,EventHandles,FALSE,INFINITE);

---

And only that declaration in a completely different source file and unrelated function. Can anyone explain why this may happen? (I'll put more code up if this isn't enough information.)

Perhaps you're debugging an old version of the program? If the source file is more recent than the executable, it will often tell you the wrong line numbers.

Also, if you have optimisation enabled, it will do strange things, jumping around between random-seeming lines.

Of course, you're probably familiar with both of these points, but I can't think of anything else at the moment. Perhaps you might want to post a bit more code, or better yet a debugger session (if you're using a command-line debugger).

Yeah, I tried rebuilding and different optimizations and neither worked.

Code:

---

PortThread[3]= AfxBeginThread(StartCryptoPortThread, &Port[3],THREAD_PRIORITY_NORMAL,0,CREATE_SUSPENDED);

UINT C_Control::StartCryptoPortThread( LPVOID pParam )
{
        char*        buffer;
        int                size = 0;
    C_RS_422_Port* pObject = ( C_RS_422_Port*)pParam;

    if (pObject == NULL )
                return 1;  // if pObject is not valid
        else
        {
                //C_Control::GetOutboundMessage(buffer,size);
               
                pObject->Send_Crypto_Key_1();    // do something with 'pObject'

                return 0;  // thread completed successfully
        }
}

void Crypto_Dlg::OnBnClickedCryptoPortAct()
{
        CString  sString;
        CButton* pButton;
        const bool Active = TRUE;

        pButton = (CButton*)GetDlgItem(IDC_Crypto_Port_Act);
        pButton->GetState();
        if (pButton->GetCheck())
        {
                pButton->SetCheck(0);
                pButton->SetState(0);
                GetDlgItemText(IDC_Crypto_Port_Act,sString);
                if (!sString.Compare("Activate"))
                {                       
                        Executive.Set_Activate(3, Active);
                        SetDlgItemText(IDC_Crypto_Port_Act,"Deactivate");
                }
                else
                {
                        Executive.Set_Activate(3,!Active);
                        SetDlgItemText(IDC_Crypto_Port_Act,"Activate");
                }
                m_b_Crypto_Port_Act = TRUE;
        }

        if (m_b_Crypto_Port_Act)
        {
                Executive.On_Crypto_Apply_Event();
                m_b_Crypto_Port_Act = FALSE;
        }
       
        if(!Executive.b_Port_Open[3] && ((CButton* )GetDlgItem(IDC_Crypto_Port_Act))->GetCheck()  )
        {
                Executive.OpenPort(3, m_Port_Number);
        }

//        UpdateData(TRUE);
//        UpdateWindow();
}

void C_Control::On_Crypto_Apply_Event()
{
        // this function controls the thread starting and stopping.
        // the threads are synchronized on a start and a stop event.
        // the thread functions that this functions controls are
        // those that write data to the ports and to the log files
       

       
                // if port is currently inactive - start and set timing
                if (!b_Port_Status[3])
                {
                        goto startports;
                }

                else // port already active
                {
                        Activate_Crypto_Port(4, FALSE);
                        goto startports;
                }
                startports:
                Activate_Crypto_Port(4, TRUE);
                Port[3].StartTransmission.SetEvent();

}

bool C_Control::Activate_Crypto_Port(int PortNumber, bool ActiveYN)
{
        // this function starts or suspends a port thread depending
        // on the state  of ActiveYN and opens the log files the first
        // time a port is activated
        static int Times_Through[4] = {0,0,0,0};
       
        DWORD SuspendCount;
        if (ActiveYN) //activate
        {
                if (Times_Through[PortNumber-1]==0)
                {
                        // Open the log files only the first time a write event happens
//                        Port[PortNumber-1].OpenLogFiles(FileNames[PortNumber-1]);
                        Times_Through[PortNumber-1]=1;
                }

                SuspendCount=PortThread[PortNumber-1]->ResumeThread();
                //code based on SuspendCount
       
}

UINT C_RS_422_Port::Send_Crypto_Key_1()
{
        DWORD aword = ::WaitForMultipleObjects(2,EventHandles,FALSE,INFINITE);
//more code
}

---

There's all the relevant code without going too overboard. Also, the first time SuspendCount is declared in this section, it will be assigned a valid number.

> PortThread[3]= AfxBeginThread(StartCryptoPortThread, &Port[3],THREAD_PRIORITY_NORMAL,0,CREATE_SUSPENDED);

> C_RS_422_Port* pObject = ( C_RS_422_Port*)pParam;
Put a breakpoint in the thread function, and compare the red bits.
If they're different, then the object you thought you were pointing at, is not what you expected.

You have to remember that all class member functions get passed a hidden 'this' pointer.
You also have to remember that the thread library doesn't know this.

Class member functions which get passed (as a pointer to a function) to say a thread library need to be declared static to suppress the 'this' pointer.

Do you have a global StartCryptoPortThread function?
If not, this should not compile.

It won't let me get to pObject in the debugger. I don't think thats the problem though, the pre-existing code that I'm trying to replicate and expand upon seems to handle that part just fine.

Then perhaps you should compile it as Debug instead of Release? That is a typical problem when you use Release.

I'm not quite sure what the difference is between the two, the problem is I can't step into the StartCryptoPortThread function from the declaration and when i put a breakpoint in the function itself, it never gets there.

Debug -> build used for debugging.
Release -> build shipped to customers (finished build).
For one thing, debug builds disable optimizations.

Okay, that's what I figured, but how do I distinguish between the two when compiling? All I've been doing is hitting F5 or the Play button next to "Debug."

Oh, now I see the drop down box, yeah, I was using Debug, not Release.

PortThread[3]= AfxBeginThread(StartCryptoPortThread, &Port[3],THREAD_PRIORITY_NORMAL,0,CREATE_SUSPENDED);

You should make sure this is being called from a function somewhere, and you are 100% sure that it is being called before resuming the thread, and that the call is succeeding. This seems like the most likely culprit, since it most closely resembles what might be expected from "undefined behaviour" as pertaining to OS thread scheduling. Without more clues to go by though, it's hard to say.

SuspendCount=PortThread[PortNumber-1]->ResumeThread();

I assume you have verified that PortNumber is indeed 4 at this line of code, and that PortThread[3] is a valid value?

PortThread[3]= AfxBeginThread(StartCryptoPortThread, &Port[3],THREAD_PRIORITY_NORMAL,0,CREATE_SUSPENDED);

That is being defined in the constructor of the Port[] instances. And yes, PortNumber is indeed 4.

*shrug*

>and that PortThread[3] is a valid value?
Not meaning to be nitpicky, but have you put a breakpoint in the constructor to ensure that the thread creation is not failing? Also, you'll want to ensure that the value of PortThread[3] doesn't change between the constructor and Activate_Crypto_Thread(). Could be some random piece of code in this 'old version' is reassigning that without you realizing it.

One other thing, along the same lines - You'll want to doublecheck that the handles in EventHandles are being properly initialized, and remain initialized at the time when Send_Crypto_Key_1() is being called.

I still think it sounds like something's uninitialized, or become de-initialized, before calling an OS scheduling function somewhere.

What is the scope of Port?
Does it last at least as long as all the threads?

Have you taken into account (see earlier posts) that the thread API cannot return a proper 'this' pointer to a C++ class member function?

Post your latest code.

To Hunter2,
Yes, I've checked the thread creation and I get valid values similar too other PortThreads that work fine.

PortThread[3] doesn't change between the constructor to Activate_Crypto_Thread the first time. Curiously, upon calling Activate_Crypto_Thread the second or third times, it inexplicably changes. In fact it almost seems to corrupt itself getting values that simply don't make sense. I've checked the whole solution for other redefinitions but to no avail.

The EventHandles are initialized properly, and are unchanged at the call of Send_Crypto_Key_1, however the code'll crash at the declaration of aword so I can't say for sure whether they stay like that.

To Salem,
Port does have the same scope as the threads.

Unfortunately I'm very new to threads and don't know how to take into account the thread API not returing a proper 'this' pointer to a C++ class member function.

Code, by in large, has remained unchanged with respect to this particular problem, however, here's the entirety of the Activate_Crypto_Key function as that seems to be where it's messing up (with the PortThread redefining itself).

Code:

---

bool C_Control::Activate_Crypto_Port(int PortNumber, bool ActiveYN)
{
        // this function starts or suspends a port thread depending
        // on the state  of ActiveYN and opens the log files the first
        // time a port is activated
        static int Times_Through[4] = {0,0,0,0};
       
        DWORD SuspendCount;
//        PortThread[3]= AfxBeginThread(StartCryptoPortThread, &Port[3],THREAD_PRIORITY_NORMAL,0,CREATE_SUSPENDED);
        if (ActiveYN) //activate
        {
                if (Times_Through[PortNumber-1]==0)
                {
                        // Open the log files only the first time a write event happens
//                        Port[PortNumber-1].OpenLogFiles(FileNames[PortNumber-1]);
                        Times_Through[PortNumber-1]=1;
                }

                SuspendCount=PortThread[PortNumber-1]->ResumeThread();
                if (SuspendCount < 2)
                {
                        b_Port_Status[PortNumber-1] = ActiveYN;
                        return(TRUE);
                }
                else
                {
                        b_Port_Status[PortNumber-1] = FALSE;
                        AfxMessageBox("Unable to resume Crypto thread", MB_OK|MB_ICONEXCLAMATION , 0);
                        return(FALSE);
                }
        }
        else       
        {
                SuspendCount=PortThread[PortNumber-1]->SuspendThread();
                if (SuspendCount >= 0)
                {
                        b_Port_Status[PortNumber-1] = ActiveYN;
                        return(TRUE);
                }
                else
                {
                        b_Port_Status[PortNumber-1] = FALSE;
                        AfxMessageBox("Unable to suspend Crypto thread", MB_OK|MB_ICONEXCLAMATION , 0);
                        return(FALSE);
                }
        }
       
}

---

Also, the 3 in
Executive.Set_Activate(3,!Active);
and
Executive.Set_Activate(3,Active);
has rightfully been changed to a 4.

Also, Send_Crypto_Key_1() and it's (previously unposted) brother Send_Crypto_Key_2() have been consolidated into a single Send_Crypto_Key(int key)

Also, since the problem has been narrowed down to and the original "odd declaration" seems to be connected to whether there were breakpoints at those locations are during debugging, I've posted a new topic ("CWinThread messing itself up") to focus in on the problem.

> Unfortunately I'm very new to threads and don't know how to take into account the thread API not returing a proper 'this' pointer to a C++ class member function.
I told you above.

Here is an example.
http://cboard.cprogramming.com/showt...t=class+static
Scroll down to posts #4 and #5

>>UINT C_Control::StartCryptoPortThread( LPVOID pParam )
Is this declared as a static function? If so, you should have nothing to worry about, because static functions act essentially the same way as global functions. There is only a problem if it is a member function (non-static). I would expect you to get an error, or at least a warning, if this is the problem though...

If I understand correctly, it seems Salem and Hunter2 are asking the same basic question right now, and the answer is yes,
>>UINT C_Control::StartCryptoPortThread( LPVOID pParam )
is declared static. Is that what you were referring to Salem?

Dunno, I can't see your screen from here, and 1 line isn't enough to extrapolate from.

Post a block of code similar to your post #3

Similar how? As in the same functions, just updated? Or are there other functions required?

Similar in scope - that is, something we can see enough of to gauge what you're doing.

But, what I posted in #3 is what I'm doing...

What you were doing in post #3 was wrong (in detail at least).

What we need to see is what it looks like NOW, following all the edits you've (allegedly) made to the code.

Simply telling us that "you've done it" just isn't good enough, because a lot of the time we find that what was said isn't what happened, or it was mis-interpreted or (well, you get the idea).

Code:

---

UINT C_Control::StartCryptoPortThread( LPVOID pParam )
{
        char*        buffer;
        int                size = 0;
    C_RS_422_Port* pObject = ( C_RS_422_Port*)pParam;

    if (pObject == NULL )
                return 1;  // if pObject is not valid
        else
        {
                //C_Control::GetOutboundMessage(buffer,size);
               
                pObject->Send_Crypto_Key(1);    // do something with 'pObject'

                return 0;  // thread completed successfully
        }
}

C_Control::C_Control()
{
        PortThread[3]= AfxBeginThread(StartCryptoPortThread, &Port[3],THREAD_PRIORITY_NORMAL,0,CREATE_SUSPENDED);
        PortThreadCrypto = PortThread[3];

        b_Port_Status[3]=FALSE;
        b_Port_Open[3] = FALSE;
}

void Crypto_Dlg::OnBnClickedCryptoPortAct()
{
        // TODO: Add your control notification handler code here
        CString  sString;
        CButton* pButton;
        const bool Active = TRUE;

        if(!Executive.b_Port_Open[3] && ((CButton* )GetDlgItem(IDC_Crypto_Port_Act))->GetCheck()  )
        {
                Executive.OpenPort(3, 7);
        }

        pButton = (CButton*)GetDlgItem(IDC_Crypto_Port_Act);
        pButton->GetState();
        if (pButton->GetCheck())
        {
                pButton->SetCheck(0);
                pButton->SetState(0);
                GetDlgItemText(IDC_Crypto_Port_Act,sString);
                if (!sString.Compare("Activate"))
                {                       
                        Executive.Set_Activate(4, Active);
                        SetDlgItemText(IDC_Crypto_Port_Act,"Deactivate");
                }
                else
                {
                        Executive.Set_Activate(4,!Active);
                        SetDlgItemText(IDC_Crypto_Port_Act,"Activate");
                }
                Executive.On_Crypto_Apply_Event();
//                m_b_Crypto_Port_Act = TRUE;
        }
}

void C_Control::On_Crypto_Apply_Event()
{
        // this function controls the thread starting and stopping.
        // the threads are synchronized on a start and a stop event.
        // the thread functions that this functions controls are
        // those that write data to the ports and to the log files
       

       
                // if port is currently inactive - start and set timing
                if (!b_Port_Status[3])
                {
                        goto startports;
                }

                else // port already active
                {
                        Activate_Crypto_Port(4, FALSE);
                        goto startports;
                }
                startports:
                Activate_Crypto_Port(4, TRUE);
                Port[3].StartTransmission.SetEvent();

}

bool C_Control::Activate_Crypto_Port(int PortNumber, bool ActiveYN)
{
        // this function starts or suspends a port thread depending
        // on the state  of ActiveYN and opens the log files the first
        // time a port is activated
        static int Times_Through[4] = {0,0,0,0};
       
        DWORD SuspendCount;
        if (ActiveYN) //activate
        {
                if (Times_Through[PortNumber-1]==0)
                {
                        // Open the log files only the first time a write event happens
//                        Port[PortNumber-1].OpenLogFiles(FileNames[PortNumber-1]);
                        Times_Through[PortNumber-1]=1;
                }

                SuspendCount=PortThread[PortNumber-1]->ResumeThread();
                if (SuspendCount < 2)
                {
                        b_Port_Status[PortNumber-1] = ActiveYN;
                        return(TRUE);
                }
                else
                {
                        b_Port_Status[PortNumber-1] = FALSE;
                        AfxMessageBox("Unable to resume Crypto thread", MB_OK|MB_ICONEXCLAMATION , 0);
                        return(FALSE);
                }
        }
        else       
        {
                SuspendCount=PortThread[PortNumber-1]->SuspendThread();
                if (SuspendCount >= 0)
                {
                        b_Port_Status[PortNumber-1] = ActiveYN;
                        return(TRUE);
                }
                else
                {
                        b_Port_Status[PortNumber-1] = FALSE;
                        AfxMessageBox("Unable to suspend Crypto thread", MB_OK|MB_ICONEXCLAMATION , 0);
                        return(FALSE);
                }
        }
       
}

UINT C_RS_422_Port::Send_Crypto_Key(int key)
{
        DWORD aword = ::WaitForMultipleObjects(2,EventHandles,FALSE,INFINITE);
       
//more code based on aword
}

---

As you can see, I haven't made many significant changes.

This is not quite related to your current problem, but I found this rather odd:

Code:

---

    // if port is currently inactive - start and set timing
    if (!b_Port_Status[3])
    {
        goto startports;
    }
    else // port already active
    {
        Activate_Crypto_Port(4, FALSE);
        goto startports;
    }
startports:
    Activate_Crypto_Port(4, TRUE);

---

as it seems like a complicated way of writing:

Code:

---

    // if port is already active - stop
    if (b_Port_Status[3])
    {
        Activate_Crypto_Port(4, FALSE);
    }

    // start and set timing
    Activate_Crypto_Port(4, TRUE);

---

Yeah, I agree, but I'm not the first one to get my hands on this code and I have to do what's already been done only change it slightly. So I've been copying and pasting a lot of code and that musta slipped through, thanks!

> PortThread[3]= AfxBeginThread(StartCryptoPortThread, &Port[3],THREAD_PRIORITY_NORMAL,0,CREATE_SUSPENDED);
So where's StartCryptoPortThread ?

Whoops, sorry. I accidentally put in StartGPSPortThread, but I've editted the post to include StartCryptoPortThread.

And is that declared as static, like I mentioned weeks ago?

Did you compare &Port[3] with pParam (again, mentioned ages ago) to see if both addresses are the same?

Guess not....

Quote:

---

Originally Posted by LowlyIntern

It won't let me get to pObject in the debugger. I don't think thats the problem though, the pre-existing code that I'm trying to replicate and expand upon seems to handle that part just fine.

---

I don't see how they wouldn't seeing as how it's a local object, it's declared in the function, and is only used once.

Quote:

---

Originally Posted by LowlyIntern

If I understand correctly, it seems Salem and Hunter2 are asking the same basic question right now, and the answer is yes,
>>UINT C_Control::StartCryptoPortThread( LPVOID pParam )
is declared static. Is that what you were referring to Salem?

---

Hey guys, thanks for all of your help and all, but one of my co-workers figured it out and it's all good now. The error resided in the code

Code:

---

UINT C_RS_422_Port::Send_Crypto_Key(int key)
{
        DWORD aword = ::WaitForMultipleObjects(2,EventHandles,FALSE,INFINITE);
       
        char Crypto_Key_1[17] = "123456789ABCDEFG";
        char Crypto_Key_2[17] = "HIJKLMNOPQRSTUVW";
        char Crypto_Key_Reversed[17];
        char Crypto_Next_Byte_Confirm;
        UINT Crypto_Key_Character = 0;

        if(key==1)
                for (int count=0; count<16; count++)
                {
                        Crypto_Key_Character += (1 & (UINT) Crypto_Key_1[count]) << 7;
                        Crypto_Key_Character += (2 & (UINT) Crypto_Key_1[count]) << 5;
                        Crypto_Key_Character += (4 & (UINT) Crypto_Key_1[count]) << 3;
                        Crypto_Key_Character += (8 & (UINT) Crypto_Key_1[count]) << 1;
                        Crypto_Key_Character += (16 & (UINT) Crypto_Key_1[count]) >> 1;
                        Crypto_Key_Character += (32 & (UINT) Crypto_Key_1[count]) >> 3;
                        Crypto_Key_Character += (64 & (UINT) Crypto_Key_1[count]) >> 5;
                        Crypto_Key_Character += (128 & (UINT) Crypto_Key_1[count]) >> 7;

                        Crypto_Key_Reversed[count] = Crypto_Key_Character;
                        Crypto_Key_Character=0;
                }
        else
                for (int count=0; count<16; count++)
                {
                        Crypto_Key_Character += (1 & (UINT) Crypto_Key_2[count]) << 7;
                        Crypto_Key_Character += (2 & (UINT) Crypto_Key_2[count]) << 5;
                        Crypto_Key_Character += (4 & (UINT) Crypto_Key_2[count]) << 3;
                        Crypto_Key_Character += (8 & (UINT) Crypto_Key_2[count]) << 1;
                        Crypto_Key_Character += (16 & (UINT) Crypto_Key_2[count]) >> 1;
                        Crypto_Key_Character += (32 & (UINT) Crypto_Key_2[count]) >> 3;
                        Crypto_Key_Character += (64 & (UINT) Crypto_Key_2[count]) >> 5;
                        Crypto_Key_Character += (128 & (UINT) Crypto_Key_2[count]) >> 7;

                        Crypto_Key_Reversed[count] = Crypto_Key_Character;
                        Crypto_Key_Character=0;
                }

        for (int count=0; count<16; count++)
        {
                if(b_Crypto_ON)
                {
                        Send_Bytes(&Crypto_Key_Reversed[count],1);
                        OutByteBufferCryptoReversed[count] = Crypto_Key_Reversed[count];
                        if(key==1)
                        {
                                OutByteBufferCrypto[count] = Crypto_Key_1[count];
                                fprintf(pFile, "%2d \t %6d ", (UINT) Crypto_Key_Reversed[count], (UINT) Crypto_Key_1[count]);
                        }
                        else
                        {
                                OutByteBufferCrypto[count] = Crypto_Key_2[count];
                                fprintf(pFile, "%2d \t %6d ", (UINT) Crypto_Key_Reversed[count], (UINT) Crypto_Key_2[count]);
                        }
                        fprintf(pFile, "\n");
                }
                else
                        count = 16;
        }

        if(Executive.GetLogCrypto())
        {
                WriteCryptoTextFile();
        }

}

---

Funnily enough, I posted that function but only included the header and the first declaration, where the problem appeared to be. However it should look like this:

Code:

---

UINT C_RS_422_Port::Send_Crypto_Key(int key)
{
        DWORD aword = ::WaitForMultipleObjects(2,EventHandles,FALSE,INFINITE);
       
        char Crypto_Key_1[17] = "123456789ABCDEFG";
        char Crypto_Key_2[17] = "HIJKLMNOPQRSTUVW";
        char Crypto_Key_Reversed[17];
        char Crypto_Next_Byte_Confirm;
        UINT Crypto_Key_Character = 0;

       
        if(key==1)
                for (int count=0; count<16; count++)
                {
                        Crypto_Key_Character += (1 & (UINT) Crypto_Key_1[count]) << 7;
                        Crypto_Key_Character += (2 & (UINT) Crypto_Key_1[count]) << 5;
                        Crypto_Key_Character += (4 & (UINT) Crypto_Key_1[count]) << 3;
                        Crypto_Key_Character += (8 & (UINT) Crypto_Key_1[count]) << 1;
                        Crypto_Key_Character += (16 & (UINT) Crypto_Key_1[count]) >> 1;
                        Crypto_Key_Character += (32 & (UINT) Crypto_Key_1[count]) >> 3;
                        Crypto_Key_Character += (64 & (UINT) Crypto_Key_1[count]) >> 5;
                        Crypto_Key_Character += (128 & (UINT) Crypto_Key_1[count]) >> 7;

                        Crypto_Key_Reversed[count] = Crypto_Key_Character;
                        Crypto_Key_Character=0;
                }
        else
                for (int count=0; count<16; count++)
                {
                        Crypto_Key_Character += (1 & (UINT) Crypto_Key_2[count]) << 7;
                        Crypto_Key_Character += (2 & (UINT) Crypto_Key_2[count]) << 5;
                        Crypto_Key_Character += (4 & (UINT) Crypto_Key_2[count]) << 3;
                        Crypto_Key_Character += (8 & (UINT) Crypto_Key_2[count]) << 1;
                        Crypto_Key_Character += (16 & (UINT) Crypto_Key_2[count]) >> 1;
                        Crypto_Key_Character += (32 & (UINT) Crypto_Key_2[count]) >> 3;
                        Crypto_Key_Character += (64 & (UINT) Crypto_Key_2[count]) >> 5;
                        Crypto_Key_Character += (128 & (UINT) Crypto_Key_2[count]) >> 7;

                        Crypto_Key_Reversed[count] = Crypto_Key_Character;
                        Crypto_Key_Character=0;
                }

        switch (aword)
        {
        case 0:       
                {
                do{
                g_CritSec.Lock();
                for (int count=0; count<16; count++)
                {
        //                Read_Bytes(&Crypto_Next_Byte_Confirm,1);

        //                if((b_Crypto_ON) & (Crypto_Next_Byte_Confirm==1))
                        if(b_Crypto_ON)
                        {
                                Send_Bytes(&Crypto_Key_Reversed[count],1);
                                OutByteBufferCryptoReversed[count] = Crypto_Key_Reversed[count];
                                if(key==1)
                                {
                                        OutByteBufferCrypto[count] = Crypto_Key_1[count];
                                        fprintf(pFile, "%2d \t %6d ", (UINT) Crypto_Key_Reversed[count], (UINT) Crypto_Key_1[count]);
                                }
                                else
                                {
                                        OutByteBufferCrypto[count] = Crypto_Key_2[count];
                                        fprintf(pFile, "%2d \t %6d ", (UINT) Crypto_Key_Reversed[count], (UINT) Crypto_Key_2[count]);
                                }
                                fprintf(pFile, "\n");
                        }
                        else
                                count = 16;
                }

                if(Executive.GetLogCrypto())
                {
                        WriteCryptoTextFile();
                }

                fprintf(pFile, "\n");
                }
                while(true);
                break;
                }
        case 1:
                {
                        return(0);
                        break;
                }
        default:
                return(0);

        }
}

---

I guess there needs to be that do while true loop to keep the thread going infinitely. Without it, the thread breaks and trying to use it again fails. My bad.