Try to loop this hash function

Question

1.00/5 (2 votes)

See more:

My updated code, however pressing the button still generates one at a time and does not generate all of them automatically. Code updated.

C#

private void openToolStripMenuItem_Click(object sender, EventArgs e)
       {
           if (dlgDirectoryBrowser.ShowDialog() == DialogResult.OK)
           {
               ImageList smallImageArray = new ImageList();
               int smallImageArray_Index = 0;
               listDetailedResults.SmallImageList = smallImageArray;

               try
               {
                   listDetailedResults.Items.Clear();
                   string[] fileEntriesFromDirectoryBrowser = System.IO.Directory.GetFiles(dlgDirectoryBrowser.SelectedPath);

                   foreach (string fileEntry in fileEntriesFromDirectoryBrowser)
                   {
                       Properties.Settings.Default.fileSizeBeforeConversion = new System.IO.FileInfo(fileEntry).Length;
                       Icon fileEntryIcon = Icon.ExtractAssociatedIcon(fileEntry);

                       smallImageArray.Images.Add(fileEntryIcon);

                       listDetailedResults.Items.Add(System.IO.Path.GetFileName(fileEntry).ToLower(), smallImageArray_Index);
                       listDetailedResults.Items[smallImageArray_Index].SubItems.Add(GetSizeReadable(Properties.Settings.Default.fileSizeBeforeConversion));
                       smallImageArray_Index++;
                   }

                   foreach (ColumnHeader columns in listDetailedResults.Columns)
                       columns.Width = -2;
               }
               catch (Exception ex) { }
               Properties.Settings.Default.count = 0;
               Properties.Settings.Default.Save();
           }
       }

       public static string GetSizeReadable(long i)
       {
           string sign = (i < 0 ? "-" : "");
           double readable = (i < 0 ? -i : i);
           string suffix;

           if (i >= 0x40000000) // Gigabyte
           {
               suffix = "GB";
               readable = (double)(i >> 20);
           }
           else if (i >= 0x100000) // Megabyte
           {
               suffix = "MB";
               readable = (double)(i >> 10);
           }
           else if (i >= 0x400) // Kilobyte
           {
               suffix = "KB";
               readable = (double)i;
           }
           else
           {
               return i.ToString(sign + "0 B"); // Byte
           }

           readable = readable / 1024;

           return sign + readable.ToString("0.## ") + suffix;
       }

       private void bgw_Md5_DoWork(object md5_sender, DoWorkEventArgs md5_e)
       {
           byte[] buffer;
           byte[] oldBuffer;
           int bytesRead;
           int oldBytesRead;
           long size;
           long totalBytesRead = 0;

           using (System.IO.Stream stream = System.IO.File.OpenRead((string)md5_e.Argument))
           using (System.Security.Cryptography.HashAlgorithm Hash_md5_Algorithm = MD5.Create())
           {
               size = stream.Length;
               buffer = new byte[4096];
               bytesRead = stream.Read(buffer, 0, buffer.Length);
               totalBytesRead += bytesRead;
               do
               {
                   oldBytesRead = bytesRead;
                   oldBuffer = buffer;

                   buffer = new byte[4096];
                   bytesRead = stream.Read(buffer, 0, buffer.Length);

                   totalBytesRead += bytesRead;

                   if (bytesRead == 0)
                   {
                       Hash_md5_Algorithm.TransformFinalBlock(oldBuffer, 0, oldBytesRead);
                   }
                   else
                   {
                       Hash_md5_Algorithm.TransformBlock(oldBuffer, 0, oldBytesRead, oldBuffer, 0);
                   }

                   bgWorkerDigest5.ReportProgress((int)((double)totalBytesRead * 100 / size));

               } while (bytesRead != 0);
               md5_e.Result = Hash_md5_Algorithm.Hash;
           }
       }

       private void bgw_Md5_RunWorkerCompleted(object md5_sender, RunWorkerCompletedEventArgs md5_e)
       {
           StringBuilder sb;
           sb = new StringBuilder();
           //pbBar.Visible = false;

           foreach (byte b in (byte[])md5_e.Result)
           {
               sb.AppendFormat("{0:X2}", b);
           }

           ListViewItem update_lvi = listDetailedResults.Items[Properties.Settings.Default.count];
           update_lvi.SubItems.Add(sb.ToString().ToLower());

           //Resize Columns For Each Listview Item
           foreach (ColumnHeader columns in listDetailedResults.Columns)
               columns.Width = -2;
           listDetailedResults.Enabled = true;
           listDetailedResults.Focus();
           Properties.Settings.Default.count++;

       }

       private void startToolStripMenuItem_Click(object sender, EventArgs e)
       {
           foreach (ListViewItem item in listDetailedResults.Items)
           {
               if (!bgWorkerDigest5.IsBusy)
                   bgWorkerDigest5.RunWorkerAsync(dlgDirectoryBrowser.SelectedPath + "\\" + item.Text);

           }
       }

Posted 30-Nov-15 22:29pm

Herboren

Updated 1-Dec-15 7:08am

v5

Add a Solution

Comments

Richard MacCutchan 1-Dec-15 9:07am

it says something along the lines of Thread being busy.
What does it actually say, and where?

Herboren 1-Dec-15 13:09pm

Code updated in question.

2 solutions

Solution 1

I've found hash calculation only in one method bgw_Md5_DoWork; and it's not shown how it's called. The implementation suggests that you call it in a separate non-UI thread. Even though some loop is shown, there is only one object which receives the hash value, md5_e.Result. Even if it wasn't outside of the loop, it would be only one. I don't know what do you mean by "all of them", but you are not trying to calculate more than one hash value. If you need more than one hash value, you need some collection which works as a sink for multiple hash values. As a single hash value is array of bytes, byte [], the data type for the data holding several hash value should be some collection of arrays of arrays of bytes, or array of array of bytes, which would be less suitable, because you may not know total number of values in advance. This is just one thing you have to do.

Now, let's look at two methods you call in the loop. Even though your code does not comprehensively represent the problem, it's very unlikely that it can be anything but System.Security.Cryptography.TransformBlock and System.Security.Cryptography.TransformFinalBlock:
https://msdn.microsoft.com/en-us/library/system.security.cryptography.hashalgorithm.transformblock%28v=vs.110%29.aspx[^],
https://msdn.microsoft.com/en-us/library/system.security.cryptography.hashalgorithm.transformblock%28v=vs.110%29.aspx[^].

First method places data in output buffer, and the last one simply returns array of bytes. Your buffers are local variables, so you don't keep any data you would collect in output buffer on the method return. As to the second method, you simply ignore its return. No wonder you loose the hash data. So, if you need this data, you should not ignore the return, and need to save it in some data collection(s) object(s) created outside of this function; you will need to pass the collection reference to your method.

Also, if you really use non-UI thread for the operation, you may have some difficulty passing data into and our of the data of your methods called in this thread. For this purpose, look at my thread wrapper technique I've suggested and widely use. Everything is explained in my past answers:
Change parameters of thread (producer) after it is started[^],
How to pass ref parameter to the thread[^],
Classes in list updating in their own thread[^],
AsyncCallback and Threadings[^] (with fully operational sample),
MultiThreading in C#[^].

—SA

Posted 1-Dec-15 7:50am

Sergey Alexandrovich Kryukov

Updated 1-Dec-15 7:52am

v3

Comments

Richard Deeming 1-Dec-15 14:19pm

From the methods and events used, I suspect the OP is using a BackgroundWorker[^], which is what's calling the bgw_Md5_DoWork event handler.

I also suspect it's the outer loop triggering the background work which is the problem, and not the loop within the DoWork event handler.

Sergey Alexandrovich Kryukov 1-Dec-15 14:22pm

Sure, agree. I actually mentioned "the implementation suggests that you call it in a separate non-UI thread", just did not want to mention BackgroundWorker specifically.
However, even the problems I spotted in the available part of code are troublesome enough and should be fixed.
—SA

Richard Deeming 1-Dec-15 14:24pm

I don't know who down-voted your answer; it's a decent answer as always. I've countered with a 5.

Sergey Alexandrovich Kryukov 1-Dec-15 14:37pm

Thank you very much, even though you don't have to "counter", I hope you just see how my advice could be useful.

I guess, this is another "hatred voting" season, which comes from time to time: one or a couple of members come on and vote 1 on just few answers in a row within the same second or few, which is clearly seen by timing; and such behavior is observed during several days. I never understood any rationale behind such patterns, sometimes it's done just on too short or too long answers, or maybe answers containing some judgement, or sometimes just first answers of a day... Pretty funny.

—SA

Herboren 1-Dec-15 15:13pm

No, not at all. No hatred here friend. However I have seen some of your posts towards most of these people, and your enthusiasm seems a bit abrasive. I know you're an intelligent individual but most of time we are looking for simple answers and the only reason I say this is the level of intelligence that some people come in here with. Sometimes the syntax you explain is beyond our reach; in another sense, the key wording you use is beyond what our teacher is showing us. Those of us or are still in fundamental training in community college or fresh into a University, are just being exposed to this content.

I try to help people with fundamentals in /r/csharp but I don't want to coin their level of intelligence by throwing them something that their teachers haven't even offered them, so I have to work from a lower subject level with them and slowly migrate their way into something modern. I hope you smell my drift.

I will have you know I don't down vote because it really is depressing being down-voted. I will however select a solution, I won't lie. If star ratings are considered down voting, I am very sorry that wasn't the intention, I only mark it 5 stars because I know this issue is common with this particular snippet. So if anyone decides to stop and search code project, I'm hoping this thread will appear to help them before they ask. I know people are in a rush to get the answer, but some of us do stay for the fire works and it really does make the most in the end when understanding the logic behind it. I am sure your expectation of people is that they don't comeback after you offered help, this just means that select few finally listened and understand the bedrock.

So please accept my gesture as kind concern and understand the level of intelligence that you come face to face within the quick questions forum =)

Sergey Alexandrovich Kryukov 1-Dec-15 16:48pm

Sorry, my link (I put only one) above was mangled by the comment submission, so the rest of the text turned the link. Perhaps this is a new bug.
Please disregard my comment above; I cannot completely remove it, probably due to this problem. (Now it's removed. Something went wrong...)
Please read below.
—SA

Sergey Alexandrovich Kryukov 1-Dec-15 16:54pm

Glad to here it's not hatred. Thank you very much. It's just hard to spot the pattern. (However, how can you know that?)

Anyway, all the reasons you are suggesting are not reasonable at all. Look, you call "abrasive" such simple thing as the syntax "beyond the reach". Come on, such heavily sarcastic manners could be called "abrasive", and even that should be considered acceptable, as soon as no personal characteristics are discussed, only thoughts, solutions, posts, and so on. By syntax... Come on, don't you need to come back to your senses? And then, everyone is supposed to learn complete syntax of the language one uses, with no exclusions, so it's better to start earlier. Not only such solutions "beyond the rich" can be useful, they also show people what they really have to learn. Also... teachers offered... No, decent engineer always go beyond the "teachers", and all real teachers really expect that. And after all, don't those people simply ask a follow-up question? :-)

So, I do smell your drill, and that drill is highly counter-productive. Let's talk about your voting pattern. If you don't down-vote, its up to you, but your up-voting criteria are more than irrational. "I only mark it 5 stars because I know this issue is common with this particular snippet." Come on, do you really want to educate mediocre people who cannot do anything beyond some well-known pattern? But why? We already have too many of them. Such people are permanently being replaced with machine (software tools, and the like), or go into the category of real engineers capable of independent and mature thinking. Moreover, with development of technology, people get more and more time for more productive and more creative work (the counter-flow also exists, to be fair). Don't get me wrong: This is not about the current level of knowledge, this is about the derivative. I really like to help the absolute beginners and highly appreciate their attitude of learning more of the best, but not everyone wants to pay decent effort; but out discussion is not about them.

This is more about your strategy and priorities. Please don't get me wrong: the way you are talking on the topic and your honesty suggests that you are a decent person who really deserve a lot respect. Besides, every unknown person already deserves a lot of respect, a credit of respect, in advance. And it's because of this respect, I hope you won't be offended by this reference: https://en.wikipedia.org/wiki/Philistinism. No way I can assume that you personally belong to this kind of people. No way. I just want to show you the edge you should be afraid of. We all are subject of rolling down to this edge and all should be afraid of it.

Let's get back to the issues of helping people and teaching. It's very important to teach even the weakest students, critically important. The ideal of education should be: there is no non-excellent students, everyone should achieve excellence. But is the ideal. In real life, this is all about infinite patience (which I try to exercise at this moment :-). But if you take it as the absolute priority, beware: you are at risk of a great sin, as you can repel future Edison (they say, Edison was dubbed "retard" at school, I suspect because the teachers failed to understand him, and his mother intentionally misread the letter from school as if they called him "genius").

Besides, in my young years, I was impressed by the striking 4-line poem by L. Ozerov; I'll try to translate the last two lines:

Do help the talented —
The talentless will pave their ways themselves...

[Failed to format the rest of the message, to be continued...]

Sergey Alexandrovich Kryukov 1-Dec-15 16:57pm

[...Continued from the previous comment]

Many think the opposite way, but real life shows me the truth in the lines of the poet. Here and there, we always see the crowds of mediocrity, and, for this crowd, it's not enough to be the crowd, they also want everyone to go down to their level, to their comfort zone, so they try to suppress people "beyond their reach", try to "manage" them, they pave their ways to all kinds of commissions, advisers, etc., and crunch everything "beyond their reach" down to mediocrity. It's not always that bad, but I'm talking about something which does take place in real life. And the modesty and cautious approach also have their truth, but I'm talking about different thing, mediocrity as aggression, a very typical thing.

There are many ways to help people, but kicking out the comfort zone is sometimes another way to help.
And, before you get outraged with my words (which may or may not happen though), please understand that I wrote so many so polemic and controversial words only because I really appreciate your honesty and the serious and decent way you expressed those concerns.

—SA

Sergey Alexandrovich Kryukov 1-Dec-15 17:19pm

Oh, and last but no least. Please don't think I can accept all your words like "intelligent individual" or "beyond what our teacher is showing", and so on. I know for sure who I am, and I am, unfortunately by far not so good as I have to be (I hardly can explain what that "have" may possibly mean); and don't think this is a modesty (I don't think I have any) or hypocrisy (I try to avoid it by all means), this is just something I know for sure. If something have the illusion that I go way too up, perhaps this person needs better teachers. :-)
—SA

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Richard Deeming · Accepted Answer · 2015-12-01T08:12:00

Solution 2

Quote:
pressing the button still generates one at a time and does not generate all of them automatically

The result is hardly surprising. The first time through the loop, you tell the BackgroundWorker to start working on a background thread. The work takes longer than retrieving the next item from the list, so for all subsequent iterations of your loop, the IsBusy property returns true, and the work is never queued.

The simplest solution is probably to track the index of the item you're processing, and queue up the next item in the RunWorkerCompleted event:

C#

private int _currentIndex;

private void ProcessItem(int index)
{
    if (0 > index || index >= listDetailedResults.Items.Count)
    {
        listDetailedResults.Enabled = true;
        listDetailedResults.Focus();
    }
    else
    {
        _currentIndex = index;
        listDetailedResults.Enabled = false;
        bgWorkerDigest5.RunWorkerAsync(Path.Combine(dlgDirectoryBrowser.SelectedPath, listDetailedResults.Items[index].Text));
    }
}

private void startToolStripMenuItem_Click(object sender, EventArgs e)
{
    if (!bgWorkerDigest5.IsBusy)
    {
        ProcessItem(0);
    }
    else
    {
        bgWorkerDigest5.CancelAsync();
    }
}

private void bgw_Md5_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
    // If the user cancelled, stop the process:
    if (e.Cancelled) 
    {
        listDetailedResults.Enabled = true;
        listDetailedResults.Focus();
        return;
    }
    
    // If there was an error, display it and stop:
    if (e.Error != null)
    {
        MessageBox.Show(e.Error.ToString(), "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
        
        listDetailedResults.Enabled = true;
        listDetailedResults.Focus();
        return;
    }
    
    // Display the result:
    StringBuilder sb = new StringBuilder();
    foreach (byte b in (byte[])e.Result)
    {
        sb.AppendFormat("{0:x2}", b);
    }
    
    int index = _currentIndex;
    ListViewItem lvi = listDetailedResults.Items[index];
    lvi.SubItems.Add(sb.ToString());
    
    //Resize Columns For Each Listview Item
    foreach (ColumnHeader columns in listDetailedResults.Columns)
    {
        columns.Width = -2;
    }
    
    // Process the next item:
    ProcessItem(index + 1);
}

private void bgw_Md5_DoWork(object sender, DoWorkEventArgs e)
{
    using (Stream stream = File.OpenRead((string)e.Argument))
    using (HashAlgorithm md5 = MD5.Create())
    {
        // Simplified code from StackOverflow:
        // http://stackoverflow.com/a/3621316/124386
        
        long size = stream.Length;
        byte[] buffer = new byte[4096];
        long totalBytesRead = 0;
        int bytesRead;
        
        while ((bytesRead = stream.Read(buffer, 0, buffer.Length)) != 0)
        {
            totalBytesRead += bytesRead;
            md5.TransformBlock(buffer, 0, bytesRead, null, 0);
            bgWorkerDigest5.ReportProgress((int)((double)totalBytesRead * 100 / size));
            
            // If the user wants to cancel, then bail out:
            if (bgWorkerDigest5.CancellationPending) 
            {
                e.Cancel = true;
                return;
            }
        }
        
        md5.TransformFinalBlock(new byte[0], 0, 0);
        e.Result = md5.Hash;
    }

    if (bgWorkerDigest5.CancellationPending) 
    {
        e.Cancel = true;
    }
}

Posted 1-Dec-15 8:12am

Richard Deeming

Updated 3-Dec-15 8:23am

v4

Comments

Herboren 1-Dec-15 14:41pm

Your solution worked wonders and the tidying helped the code be more readable thank you =).

I was faced with this error after trying to hash multiple files ranging in the Gigabytes: Additional information: Value of '-2147483648' is not valid for 'Value'. 'Value' should be between 'minimum' and 'maximum'

from code:

private void bgw_Md5_ProgressChanged(object md5_sender, ProgressChangedEventArgs md5_e)
{
progressBackGroundWork.Value = md5_e.ProgressPercentage;
}

when hashing files larger than 1.5GB

However I am gonna assume that the value of the progress bar is not reseting and its just counting more than the maximum value.

Richard Deeming 1-Dec-15 14:47pm

Sorry, the totalBytesRead variable needs to be a long, not an int.

Herboren 1-Dec-15 15:18pm

totalBytesRead is long, still throwing.

Richard Deeming 1-Dec-15 15:24pm

That's odd - I can't make equivalent code produce a value outside of the 0-100 range.

What values have you got in size and totalBytesRead when it throws?

Herboren 1-Dec-15 15:29pm

When I set my breakpoint on

private void bgw_Md5_ProgressChanged(object md5_sender, ProgressChangedEventArgs md5_e)
{
progressBackGroundWork.Value = md5_e.ProgressPercentage;
}

I get

md5_e.ProgressPercentage = -2147483648
ProgressPercentage = -2147483648
UserState = null

Richard Deeming 1-Dec-15 15:31pm

OK, but further up the call-stack, you should have the bgw_Md5_DoWork method. If you select that, you should be able to see the values in the size and totalBytesRead variables.

Herboren 1-Dec-15 16:07pm

Its weird, when I try to break at totalbytes read anywhere in the DoWork block, it only returns the minimum of 8192, but when I step out, my 'Step' buttons grey out and wont allow me to interact with the application or step further. Any suggestions where I should break?

Richard Deeming 1-Dec-15 16:12pm

Try a conditional breakpoint. Split out the percentage into a local variable:

int percent = (int)((double)totalBytesRead * 100 / size);
bgWorkerDigest5.ReportProgress(percent);

Set a breakpoint on the ReportProgress line, right-click on the breakpoint, and set a condition for percent < 0. That should then break into the method just before you get the exception.

Herboren 1-Dec-15 16:23pm

I set the break on

bgWorkerDigest5.ReportProgress(percent);

Checked the totalbytesread value and still produced these results: Imgur: http://i.imgur.com/J0tBXfa.png

Richard Deeming 1-Dec-15 16:33pm

That's very odd. The only way I can get that to happen is if both size and totalBytesRead are 0.

But if you're using the while loop from my answer, rather than the do..while loop from your question, then the loop body wouldn't execute if that was the case.

Herboren 1-Dec-15 16:41pm

When I was tidying up the code after you minimized it, I was Ctrl+Z and undoing code so while I was undoing it cast the totalbytesread back to int from long. So I changed it back and put 2 try..catch statements in, works perfect. Problem fixed! Thank you so much =)

Herboren 3-Dec-15 14:07pm

I didnt mean to res a dead thread, but still related. But I was trying to implement a cancel button to stop the async by adding a while to the DoWork method

while (bgWorkerMD5.CancellationPending != true)
{
try
{
using (Stream stream = File.OpenRead((string)MD5_e.Argument))
using (HashAlgorithm md5 = MD5.Create())
{
long size = stream.Length;
byte[] buffer = new byte[4096];
long totalBytesRead = 0;
int bytesRead;

while ((bytesRead = stream.Read(buffer, 0, buffer.Length)) != 0)
{
totalBytesRead += bytesRead;
md5.TransformBlock(buffer, 0, bytesRead, null, 0);
bgWorkerMD5.ReportProgress((int)((double)totalBytesRead * 100 / size));
}

md5.TransformFinalBlock(new byte[0], 0, 0);
MD5_e.Result = md5.Hash;
}
}
catch (Exception ex) { }
} MD5_e.Cancel = true;

and then in my button that I used to start the DoWork is renamed to cancel with an if within

private void button1_Click(object sender, EventArgs e)
{
if (!bgWorkerMD5.IsBusy)
ProcessItem(0);
else
bgWorkerMD5.CancelAsync();
}

But the progress bar keeps updating and hashes are still being generated. I read somewhere online that dowork may have already completed which is why the cancel didnt fire. However, its hashing files in the GB which should give me enough time to cancel. But I have tried everything now.

Richard Deeming 3-Dec-15 14:26pm

The outer loop won't work - it will keep re-processing the same file over and over again until you cancel.

You need to check within the while ((bytesRead = ... loop. You'll also need to check in the bgw_Md5_RunWorkerCompleted method, so that it doesn't queue up the next item if the operation was cancelled.

I've updated my answer with the necessary changes.

Herboren 3-Dec-15 14:45pm

It worked, however it only stop to present file being generated. It then skips hashing the file and moves to the next. However I suspect ProcessItem() has a lot to do with why.

Richard Deeming 3-Dec-15 14:47pm

Did you miss the check for e.Cancelled at the start of the bgw_Md5_RunWorkerCompleted method, or the e.Cancel = true in the bgw_Md5_DoWork method?

Herboren 3-Dec-15 14:52pm

My Runworkercomplete:

private void bgw_RunWorkerCompleted(object SHA_sender, RunWorkerCompletedEventArgs rwc_e)
{
if (bgWorkerMD5.CancellationPending)
return;

if (rwc_e.Error != null)
{
MessageBox.Show(rwc_e.Error.ToString(), "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);

return;
}

StringBuilder sb;
sb = new StringBuilder();
try
{
foreach (byte b in (byte[])rwc_e.Result)
{
sb.AppendFormat("{0:X2}", b);
}
}
catch (Exception ex) { }

int index = _currentIndex;

ListViewItem update_lvi = listDetailedResults.Items[index];
update_lvi.SubItems[2].Text = (sb.ToString().ToLower());

//Resize Columns For Each Listview Item
foreach (ColumnHeader columns in listDetailedResults.Columns)
columns.Width = -2;

ProcessItem(index + 1);
menuExport.Enabled = true;

}

My DoWork

if (bgWorkerMD5.CancellationPending)
{
MD5_e.Cancel = true;
return;
}

And it just skips over the working index and moves to the next and begins to process that

Richard Deeming 3-Dec-15 14:55pm

In the RunWorkerCompleted method, you need to check rwc_e.Cancelled, not bgWorkderMD5.CancellationPending.

The CancellationPending property is reset just before the RunWorkerCompleted event is fired.

Herboren 3-Dec-15 15:05pm

Ahah! that did it. That was my fnckup sorry

Herboren 5-Dec-15 2:44am

My Sincerest Apologies. Everything works great. Now if I want to add a combo box to change the hash function what exactly would I change? The ProcessItem()?

Richard Deeming 7-Dec-15 6:35am

Change the bgw_Md5_DoWork function. Replace MD5.Create() with HashAlgorithm.Create(yourChosenAlgorithmName).

HashAlgorithm.Create Method[^]

Herboren 1-Dec-15 16:34pm

Hold on I think I fixed it. Will post back in a sec.

BillWoodruff 1-Dec-15 22:36pm

+5