search

nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Corruption to description field will result in program crash #493

Open nus-pe-bot opened 1 day ago

nus-pe-bot commented 1 day ago

If there is a corruption in description field, program will infinite loop and crash on next run. This could potentially be a big issue as if there are thousands of transactions, just one of the descriptions need to be corrupted for this to happen. It would be very hard for user to find the error manually and program cannot be relaunched until json file is either deleted or that line is mended. I gave this a severity high because program is unusable state if this happen. It might not happen because a user editing json file directly as well. It may have just been a result of file corruption.

image.png

image.png

image.png

Steps to replicate add a transaction close the program edit description to simulate corruption run the program

[original: nus-cs2113-AY2425S1/pe-interim#119] [original labels: type.FeatureFlaw severity.High]

Gao327 commented 1 day ago

Team's Response

Thanks for raising this. It is a bug, which I fail to address in the UG. The reason should be gson conversion does not take the JSON file line by line but object by object instead. I choose medium because I agree that it can be bettered but I do not think this issue will affect most users as it is not recommended to edit JSON file directly (as addressed in UG).

image.png

Duplicate status (if any):

--