Error C6385 In C++ (AES) - Programming - Linus Tech Tips

Có thể bạn quan tâm

Jump to content

Existing user? Sign In
Sign In
- Remember me Not recommended on shared computers
- Sign In
  Forgot your password?
Or sign in with one of these services
Sign in with X Sign in with Microsoft Sign in with Google Sign in with Steam Sign in with Discord
Linus Media Group is not associated with these services
Sign Up

Forums
Community Standards
All Activity
My Activity Streams
- Status Updates
- Recent Topics
More
- Blogs
- Gallery
- Events
- Downloads
- Online Users
- Staff
- Forum Subscriptions

Recent Topics

Home
Software
Programming
Error C6385 in C++ (AES)

Please Use CODE Tags Error C6385 in C++ (AES)

By Lennart van de Merwe May 7, 2020 in Programming

c++
error c6385
aes

Share https://linustechtips.com/topic/1190652-error-c6385-in-c-aes/ More sharing options... Followers 2

Lennart van de Merwe
Member

unsigned

char

decryptedmessage

new

unsigned

char

[

LenOfPaddedMessage

];

for

(

int

;

LenOfPaddedMessage

;

++)

{

decryptedmessage

[

]

encryptedmessage2

[

];

}

for

(

int

;

LenOfPaddedMessage

;

)

{

AES_Decrypt

(

decryptedmessage

inputkey

);

}

cout

endl

;

cout

"\nDecrypted message:"

endl

;

for

(

int

;

LenOfPaddedMessage

;

++)

{

cout

decryptedmessage

[

];

}

cout

endl

;

string

AnswerYesNo

;

// een test/encryptie module

cout

"\nType 'yes' if you wish to encrypt the message, answer 'no' if not: "

;

getline

(

cin

AnswerYesNo

);

int

AnswerLength

;

AnswerLength

AnswerYesNo

length

();

(

AnswerLength

)

{

int

originalLen

strlen

((

const

char

decryptedmessage

);

int

lenOfPaddedMessage

originalLen

;

(

lenOfPaddedMessage

)

lenOfPaddedMessage

(

lenOfPaddedMessage

)

;

unsigned

char

paddedMessage

new

unsigned

char

[

lenOfPaddedMessage

];

for

(

int

;

lenOfPaddedMessage

;

++)

{

(

originalLen

)

paddedMessage

[

]

;

else

paddedMessage

[

]

decryptedmessage

[

];

}

Hi guys this is Austin, I'm having a problem with my C++ project for school (it's making AES, if you're wondering). I'm a massive noob when it comes to coding, let alone C languages, so please have mercy on my soul. The problem I'm having is in the second to last line: "... else paddedMessage[x] = decryptedmessage[x];" The warning Visual gives me is coded C6385. Of course I've looked this up and I understand what the problem might be, but I can't seem to figure out the cause of it. I suspect it's something to do "decryptedmessage", but I'm not sure.The code listed above isn't the entire code (that's something like 600 lines: too much for this post I suppose, but if you need it, just ask), I've chosen to take the code from the 'birth' of decryptedmessage until the error. If you need more, once again, just ask. I'm sorry for my awful description of the problem and I know I'm a horrible noob. Please help me out, it would mean a lot. Thanks in advance.

Link to comment

https://linustechtips.com/topic/1190652-error-c6385-in-c-aes/

Share on other sites

Link to post

Share on other sites

AbydosOne
Member
Sony Fanboy

4.4k
305

9 minutes ago, Lennart van de Merwe said:
coded C6385

See here.

Basically, it's just warning you there's a chance that 'x' might be a larger value than the declared length of 'decryptedMessage'. Check that x < decryptedMessageLength in your for loop.

Network:

Spoiler

Lennart van de Merwe
1

Link to comment

https://linustechtips.com/topic/1190652-error-c6385-in-c-aes/#findComment-13573568

Share on other sites

Link to post

Share on other sites

Lennart van de Merwe
Member

Author

14 minutes ago, AbydosOne said:
See here.

Basically, it's just warning you there's a chance that 'x' might be a larger value than the declared length of 'decryptedMessage'. Check that x < decryptedMessageLength in your for loop.

I'm sorry, but what do you mean by that? I'm a noob remember X<

Link to comment

https://linustechtips.com/topic/1190652-error-c6385-in-c-aes/#findComment-13573647

Share on other sites

Link to post

Share on other sites

AbydosOne
Member
Sony Fanboy

4.4k
305

59 minutes ago, Lennart van de Merwe said:
I'm sorry, but what do you mean by that? I'm a noob remember X<

Basically, if you have something with 10 slots, and you ask it for whatever is in slot 11, the thing isn't going to know how to respond, and so it might throw an error, or crash, or give you what would be in slot 11 if it existed (this is usually the nastiest outcome for various reasons). Point is, there a chance you might ask the array for something in a position that doesn't exist, and so you need to ensure that your value of 'x' doesn't tell the arrays 'decryptedmessage' or 'paddedMessage' (in that line) to pick from a slot beyond what they have, by making sure that 'x' is always smaller than the length of either array.

Network:

Spoiler

Link to comment

https://linustechtips.com/topic/1190652-error-c6385-in-c-aes/#findComment-13573873

Share on other sites

Link to post

Share on other sites

Lennart van de Merwe
Member

Author

2 minutes ago, AbydosOne said:
Basically, if you have something with 10 slots, and you ask it for whatever is in slot 11, the thing isn't going to know how to respond, and so it might throw an error, or crash, or give you what would be in slot 11 if it existed (this is usually the nastiest outcome for various reasons). Point is, there a chance you might ask the array for something in a position that doesn't exist, and so you need to ensure that your value of 'x' doesn't tell the arrays 'decryptedmessage' or 'paddedMessage' (in that line) to pick from a slot beyond what they have, by making sure that 'x' is always smaller than the length of either array.

Yes, but that int is determined by taking the length of the decryptedmessage, which then also defines the length of the paddedmessage. Could it be something to do with:

Link to comment

https://linustechtips.com/topic/1190652-error-c6385-in-c-aes/#findComment-13573885

Share on other sites

Link to post

Share on other sites

wanderingfool2
Member

3.9k
27

5 minutes ago, Lennart van de Merwe said:
Yes, but that int is determined by taking the length of the decryptedmessage, which then also defines the length of the paddedmessage. Could it be something to do with:
int originalLen = strlen((const char*)decryptedmessage); int lenOfPaddedMessage = originalLen;

It's a compiler warning, which means that it doesn't always do the most indepth analysis of what the upper bound may be. It is giving that warning because it just detected that the number might be larger than the original (in this case it is sort of a guideline to look into to make sure it won't).

As a note though, strlen is null terminated; so yes originalLen could be greater than the defined length and you could be reading the memory that comes after the decryptedmessage.

All you could use something like as follows to catch it properly

*So now this is just my opinion, but it I always try keeping variable names distinct from each other. Yes, l vs L makes it a different variable but to me it just makes things confusing when they are used together (having to remember which is which as I don't usually process the capitals in my head)

3735928559 - Beware of the dead beef

AbydosOne
1

Link to comment

https://linustechtips.com/topic/1190652-error-c6385-in-c-aes/#findComment-13573993

Share on other sites

Link to post

Share on other sites

Lennart van de Merwe
Member

Author

10 minutes ago, wanderingfool2 said:
It's a compiler warning, which means that it doesn't always do the most indepth analysis of what the upper bound may be. It is giving that warning because it just detected that the number might be larger than the original (in this case it is sort of a guideline to look into to make sure it won't).

As a note though, strlen is null terminated; so yes originalLen could be greater than the defined length and you could be reading the memory that comes after the decryptedmessage.
int main() { char[2] a; a[0] = 'a'; a[1] = 'b'; int len = strlen(a); //Len is not well defined. In theory you might think it is 2, but in practice it could be anything //Since it will depend what is stored in the memory location a[2] (sometimes it might be 0, so you might get the right answer) //but it isn't guarenteed return 0; }

All you could use something like as follows to catch it properly
if(lenOfPaddedMessage > LenOfPaddedMessage) lenOfPaddedMessage = LenOfPaddedMessage;

*So now this is just my opinion, but it I always try keeping variable names distinct from each other. Yes, l vs L makes it a different variable but to me it just makes things confusing when they are used together (having to remember which is which as I don't usually process the capitals in my head)

Thanks a lot for the input and clear explanation. Unfortunately it only makes it worse when it comes to the output of the code, so I'm leaving the bug as it is, especially since I have to start handing in my assignment.

Link to comment

https://linustechtips.com/topic/1190652-error-c6385-in-c-aes/#findComment-13574055

Share on other sites

Link to post

Share on other sites

wanderingfool2
Member

3.9k
27

17 minutes ago, Lennart van de Merwe said:
Thanks a lot for the input and clear explanation. Unfortunately it only makes it worse when it comes to the output of the code, so I'm leaving the bug as it is, especially since I have to start handing in my assignment.

Okay, so I actually read your code...yea, you would actually want to apply the fix to originalLen...or if it allows it strnlen...and cap it there

3735928559 - Beware of the dead beef

Link to comment

https://linustechtips.com/topic/1190652-error-c6385-in-c-aes/#findComment-13574175

Share on other sites

Link to post

Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign in

Already have an account? Sign in here.

https://linustechtips.com/topic/1190652-error-c6385-in-c-aes/

Followers

Go to topic listing

Programming

Featured Topics
- Credential Stuffing Attack
  Spotty · Yesterday at 08:26 AM
  4
- I Love WU - 2026
  HoldSquat · January 8
  263
Topics
- Need mouse switch buying help
  ChristianHM · 35 minutes ago Posted in Peripherals
  0
- How to manage dependancies [arch]
  apoyusiken · 45 minutes ago Posted in Linux, macOS and Everything Not-Windows
  2
- Is the Vivobook Go 15 OLED (E1504F) a good laptop for school or is it overkill?
  ferbert72. · 2 hours ago Posted in Laptops and Pre-Built Systems
  3
- Low power GPU for linux with 3xDisplayPort for KVM-Switch
  Eardy · 2 hours ago Posted in Graphics Cards
  2
- Asus Prime Z270A i7 7700K fails to boot
  SmookinJoe · 5 hours ago Posted in Troubleshooting
  1
- Looking for gaming headset with excellent voice isolation
  NorwegianWoodBadger · 6 hours ago Posted in Audio
  5
- XMP Boot issue (help!)
  GeneDrozd · 12 hours ago Posted in Troubleshooting
  11
- steamos gaming mode macro
  apoyusiken · 13 hours ago Posted in Linux, macOS and Everything Not-Windows
  3
- Moving from macOS to Windows
  wonderland-1989 · 14 hours ago Posted in Windows
  5
- BOINC Crunchathon 2026 (March 9th - 30th)
  TheLANguy · 15 hours ago Posted in Folding@home, Boinc, and Coin Mining
  14
Join our Discord server Get LTT merch Support us on Floatplane Support the forum
play_circle_filled
Latest From Linus Tech Tips:

Are these a scam?
- Watch Now
- More Videos
- Discuss
play_circle_filled
Latest From ShortCircuit:

the BEST Webcam on the market right now
- Watch Now
- More Videos
play_circle_filled
Latest From TechLinked:

Discord is Scanning our faces now
- Watch Now
- More Videos
play_circle_filled
Latest From GameLinked:

Valve...Don't Do This
- Watch Now
- More Videos