The following document contains the results of FindBugs Report
FindBugs Version is 1.2.1
Threshold is Low
Effort is Default
Classes | Bugs | Errors | Missing Classes |
---|---|---|---|
422 | 106 | 0 | 0 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.component.image.backgroundgenerator.FileReaderRandomBackgroundGenerator.getImage(File) may fail to close stream on exception | BAD_PRACTICE | OS_OPEN_STREAM_EXCEPTION_PATH | 194 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.component.image.backgroundgenerator.MultipleShapeBackgroundGenerator.MultipleShapeBackgroundGenerator(Integer, Integer) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 54 |
Method com.octo.captcha.component.image.backgroundgenerator.MultipleShapeBackgroundGenerator.MultipleShapeBackgroundGenerator(Integer, Integer) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 59 |
Method com.octo.captcha.component.image.backgroundgenerator.MultipleShapeBackgroundGenerator.MultipleShapeBackgroundGenerator(Integer, Integer) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 64 |
Method com.octo.captcha.component.image.backgroundgenerator.MultipleShapeBackgroundGenerator.MultipleShapeBackgroundGenerator(Integer, Integer) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 69 |
Method com.octo.captcha.component.image.backgroundgenerator.MultipleShapeBackgroundGenerator.MultipleShapeBackgroundGenerator(Integer, Integer) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 74 |
Method com.octo.captcha.component.image.backgroundgenerator.MultipleShapeBackgroundGenerator.MultipleShapeBackgroundGenerator(Integer, Integer, Color, Color, Integer, Integer, Integer, Integer, Color, Color, Integer) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 54 |
Method com.octo.captcha.component.image.backgroundgenerator.MultipleShapeBackgroundGenerator.MultipleShapeBackgroundGenerator(Integer, Integer, Color, Color, Integer, Integer, Integer, Integer, Color, Color, Integer) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 59 |
Method com.octo.captcha.component.image.backgroundgenerator.MultipleShapeBackgroundGenerator.MultipleShapeBackgroundGenerator(Integer, Integer, Color, Color, Integer, Integer, Integer, Integer, Color, Color, Integer) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 64 |
Method com.octo.captcha.component.image.backgroundgenerator.MultipleShapeBackgroundGenerator.MultipleShapeBackgroundGenerator(Integer, Integer, Color, Color, Integer, Integer, Integer, Integer, Color, Color, Integer) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 69 |
Method com.octo.captcha.component.image.backgroundgenerator.MultipleShapeBackgroundGenerator.MultipleShapeBackgroundGenerator(Integer, Integer, Color, Color, Integer, Integer, Integer, Integer, Color, Color, Integer) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 74 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.component.image.color.RandomListColorGenerator.RandomListColorGenerator(Color[]) may expose internal representation by storing an externally mutable object into RandomListColorGenerator.colorsList | MALICIOUS_CODE | EI_EXPOSE_REP2 | 51 |
Bug | Category | Details | Line |
---|---|---|---|
The field name com.octo.captcha.component.image.deformation.ImageDeformationByBufferedImageOp.ImageOperations doesn't start with an lower case letter | BAD_PRACTICE | NM_FIELD_NAMING_CONVENTION | Not available |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.component.image.deformation.ImageDeformationByFilters.ImageDeformationByFilters(ImageFilter[]) may expose internal representation by storing an externally mutable object into ImageDeformationByFilters.filters | MALICIOUS_CODE | EI_EXPOSE_REP2 | 36 |
Bug | Category | Details | Line |
---|---|---|---|
Dead store to filter in com.octo.captcha.component.image.deformation.PuzzleImageDeformation.deformImage(BufferedImage) | STYLE | DLS_DEAD_LOCAL_STORE | 92 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.component.image.fontgenerator.RandomFontGenerator.getBadFontNamePrefixes() may expose internal representation by returning RandomFontGenerator.badFontNamePrefixes | MALICIOUS_CODE | EI_EXPOSE_REP | 251 |
com.octo.captcha.component.image.fontgenerator.RandomFontGenerator.RandomFontGenerator(Integer, Integer, String[]) may expose internal representation by storing an externally mutable object into RandomFontGenerator.badFontNamePrefixes | MALICIOUS_CODE | EI_EXPOSE_REP2 | 86 |
com.octo.captcha.component.image.fontgenerator.RandomFontGenerator.setBadFontNamePrefixes(String[]) may expose internal representation by storing an externally mutable object into RandomFontGenerator.badFontNamePrefixes | MALICIOUS_CODE | EI_EXPOSE_REP2 | 258 |
com.octo.captcha.component.image.fontgenerator.RandomFontGenerator.defaultBadFontNamePrefixes should be both final and package protected | MALICIOUS_CODE | MS_FINAL_PKGPROTECT | 43 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.component.image.textpaster.DecoratedRandomTextPaster.DecoratedRandomTextPaster(Integer, Integer, ColorGenerator, Boolean, TextDecorator[]) may expose internal representation by storing an externally mutable object into DecoratedRandomTextPaster.decorators | MALICIOUS_CODE | EI_EXPOSE_REP2 | 51 |
com.octo.captcha.component.image.textpaster.DecoratedRandomTextPaster.DecoratedRandomTextPaster(Integer, Integer, ColorGenerator, TextDecorator[]) may expose internal representation by storing an externally mutable object into DecoratedRandomTextPaster.decorators | MALICIOUS_CODE | EI_EXPOSE_REP2 | 38 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.component.image.textpaster.GlyphsPaster.GlyphsPaster(Integer, Integer, ColorGenerator, Boolean, GlyphsVisitors[]) may expose internal representation by storing an externally mutable object into GlyphsPaster.glyphVisitors | MALICIOUS_CODE | EI_EXPOSE_REP2 | 67 |
com.octo.captcha.component.image.textpaster.GlyphsPaster.GlyphsPaster(Integer, Integer, ColorGenerator, Boolean, GlyphsVisitors[], GlyphsDecorator[]) may expose internal representation by storing an externally mutable object into GlyphsPaster.glyphVisitors | MALICIOUS_CODE | EI_EXPOSE_REP2 | 73 |
com.octo.captcha.component.image.textpaster.GlyphsPaster.GlyphsPaster(Integer, Integer, ColorGenerator, Boolean, GlyphsVisitors[], GlyphsDecorator[]) may expose internal representation by storing an externally mutable object into GlyphsPaster.glyphsDecorators | MALICIOUS_CODE | EI_EXPOSE_REP2 | 74 |
com.octo.captcha.component.image.textpaster.GlyphsPaster.GlyphsPaster(Integer, Integer, ColorGenerator, GlyphsVisitors[]) may expose internal representation by storing an externally mutable object into GlyphsPaster.glyphVisitors | MALICIOUS_CODE | EI_EXPOSE_REP2 | 56 |
com.octo.captcha.component.image.textpaster.GlyphsPaster.GlyphsPaster(Integer, Integer, ColorGenerator, GlyphsVisitors[], GlyphsDecorator[]) may expose internal representation by storing an externally mutable object into GlyphsPaster.glyphVisitors | MALICIOUS_CODE | EI_EXPOSE_REP2 | 61 |
com.octo.captcha.component.image.textpaster.GlyphsPaster.GlyphsPaster(Integer, Integer, ColorGenerator, GlyphsVisitors[], GlyphsDecorator[]) may expose internal representation by storing an externally mutable object into GlyphsPaster.glyphsDecorators | MALICIOUS_CODE | EI_EXPOSE_REP2 | 62 |
com.octo.captcha.component.image.textpaster.GlyphsPaster.GlyphsPaster(Integer, Integer, Color, GlyphsVisitors[]) may expose internal representation by storing an externally mutable object into GlyphsPaster.glyphVisitors | MALICIOUS_CODE | EI_EXPOSE_REP2 | 51 |
com.octo.captcha.component.image.textpaster.GlyphsPaster.GlyphsPaster(Integer, Integer, GlyphsVisitors[]) may expose internal representation by storing an externally mutable object into GlyphsPaster.glyphVisitors | MALICIOUS_CODE | EI_EXPOSE_REP2 | 46 |
Bug | Category | Details | Line |
---|---|---|---|
Dead store to arbitraryHorizontalPadding in com.octo.captcha.component.image.textpaster.MutableAttributedString.moveToRandomSpot(BufferedImage, Point2D) | STYLE | DLS_DEAD_LOCAL_STORE | 145 |
Dead store to arbitraryVerticalPadding in com.octo.captcha.component.image.textpaster.MutableAttributedString.moveToRandomSpot(BufferedImage, Point2D) | STYLE | DLS_DEAD_LOCAL_STORE | 146 |
Dead store to RS in com.octo.captcha.component.image.textpaster.MutableAttributedString.toString() | STYLE | DLS_DEAD_LOCAL_STORE | 333 |
Primitive boxed just to call toString in com.octo.captcha.component.image.textpaster.MutableAttributedString.MutableAttributedString(Graphics2D, AttributedString, int) | PERFORMANCE | DM_BOXED_PRIMITIVE_TOSTRING | 94 |
Method com.octo.captcha.component.image.textpaster.MutableAttributedString.MutableAttributedString(Graphics2D, AttributedString, int) invokes inefficient Character(char) constructor; use Character.valueOf(char) instead | PERFORMANCE | DM_NUMBER_CTOR | 94 |
int division result cast to double or float in com.octo.captcha.component.image.textpaster.MutableAttributedString.moveToRandomSpot(BufferedImage, Point2D) | STYLE | ICAST_IDIV_CAST_TO_DOUBLE | 169 |
Unread field: com.octo.captcha.component.image.textpaster.MutableAttributedString.originalAttributedString | PERFORMANCE | URF_UNREAD_FIELD | 76 |
Unused field: com.octo.captcha.component.image.textpaster.MutableAttributedString.glyphVectors | PERFORMANCE | UUF_UNUSED_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.component.image.textpaster.RandomTextPaster.RandomTextPaster(Integer, Integer, Color[]) may expose internal representation by storing an externally mutable object into RandomTextPaster.textColors | MALICIOUS_CODE | EI_EXPOSE_REP2 | 38 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.component.image.textpaster.VisitedAndDecoratedTextPaster.VisitedAndDecoratedTextPaster(Integer, Integer, ColorGenerator, Boolean, TextVisitor[], TextDecorator[]) may expose internal representation by storing an externally mutable object into VisitedAndDecoratedTextPaster.textDecorators | MALICIOUS_CODE | EI_EXPOSE_REP2 | 46 |
com.octo.captcha.component.image.textpaster.VisitedAndDecoratedTextPaster.VisitedAndDecoratedTextPaster(Integer, Integer, ColorGenerator, Boolean, TextVisitor[], TextDecorator[]) may expose internal representation by storing an externally mutable object into VisitedAndDecoratedTextPaster.textVisitors | MALICIOUS_CODE | EI_EXPOSE_REP2 | 45 |
com.octo.captcha.component.image.textpaster.VisitedAndDecoratedTextPaster.VisitedAndDecoratedTextPaster(Integer, Integer, ColorGenerator, TextVisitor[], TextDecorator[]) may expose internal representation by storing an externally mutable object into VisitedAndDecoratedTextPaster.textDecorators | MALICIOUS_CODE | EI_EXPOSE_REP2 | 40 |
com.octo.captcha.component.image.textpaster.VisitedAndDecoratedTextPaster.VisitedAndDecoratedTextPaster(Integer, Integer, ColorGenerator, TextVisitor[], TextDecorator[]) may expose internal representation by storing an externally mutable object into VisitedAndDecoratedTextPaster.textVisitors | MALICIOUS_CODE | EI_EXPOSE_REP2 | 39 |
com.octo.captcha.component.image.textpaster.VisitedAndDecoratedTextPaster.VisitedAndDecoratedTextPaster(Integer, Integer, Color, TextVisitor[], TextDecorator[]) may expose internal representation by storing an externally mutable object into VisitedAndDecoratedTextPaster.textDecorators | MALICIOUS_CODE | EI_EXPOSE_REP2 | 34 |
com.octo.captcha.component.image.textpaster.VisitedAndDecoratedTextPaster.VisitedAndDecoratedTextPaster(Integer, Integer, Color, TextVisitor[], TextDecorator[]) may expose internal representation by storing an externally mutable object into VisitedAndDecoratedTextPaster.textVisitors | MALICIOUS_CODE | EI_EXPOSE_REP2 | 33 |
com.octo.captcha.component.image.textpaster.VisitedAndDecoratedTextPaster.VisitedAndDecoratedTextPaster(Integer, Integer, TextVisitor[], TextDecorator[]) may expose internal representation by storing an externally mutable object into VisitedAndDecoratedTextPaster.textDecorators | MALICIOUS_CODE | EI_EXPOSE_REP2 | 28 |
com.octo.captcha.component.image.textpaster.VisitedAndDecoratedTextPaster.VisitedAndDecoratedTextPaster(Integer, Integer, TextVisitor[], TextDecorator[]) may expose internal representation by storing an externally mutable object into VisitedAndDecoratedTextPaster.textVisitors | MALICIOUS_CODE | EI_EXPOSE_REP2 | 27 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.component.image.textpaster.glyphsdecorator.BaffleGlyphsDecorator.BaffleGlyphsDecorator(Integer, ColorGenerator) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 48 |
Method com.octo.captcha.component.image.textpaster.glyphsdecorator.BaffleGlyphsDecorator.BaffleGlyphsDecorator(Integer, Color) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 48 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.component.image.textpaster.textdecorator.BaffleTextDecorator.BaffleTextDecorator(Integer, ColorGenerator) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 47 |
Method com.octo.captcha.component.image.textpaster.textdecorator.BaffleTextDecorator.BaffleTextDecorator(Integer, Color) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 47 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.component.image.textpaster.textdecorator.LineTextDecorator.LineTextDecorator(Integer, ColorGenerator) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 38 |
Method com.octo.captcha.component.image.textpaster.textdecorator.LineTextDecorator.LineTextDecorator(Integer, Color) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 38 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.component.image.utils.ToolkitFactory.TOOLKIT_IMPL isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 30 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.component.word.DefaultSizeSortedWordList.addWord(String) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 51 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.component.word.wordgenerator.ComposeDictionaryWordGenerator.checkAndFindSmaller(String, int, Locale) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 76 |
Method com.octo.captcha.component.word.wordgenerator.ComposeDictionaryWordGenerator.getWord(Integer, Locale) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 40 |
Method com.octo.captcha.component.word.wordgenerator.ComposeDictionaryWordGenerator.getWord(Integer, Locale) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 48 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.engine.GenericCaptchaEngine.getFactories() may expose internal representation by returning GenericCaptchaEngine.factories | MALICIOUS_CODE | EI_EXPOSE_REP | 54 |
com.octo.captcha.engine.GenericCaptchaEngine.GenericCaptchaEngine(CaptchaFactory[]) may expose internal representation by storing an externally mutable object into GenericCaptchaEngine.factories | MALICIOUS_CODE | EI_EXPOSE_REP2 | 45 |
com.octo.captcha.engine.GenericCaptchaEngine.setFactories(CaptchaFactory[]) may expose internal representation by storing an externally mutable object into GenericCaptchaEngine.factories | MALICIOUS_CODE | EI_EXPOSE_REP2 | 61 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.engine.image.fisheye.SimpleFishEyeEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 74 |
Method com.octo.captcha.engine.image.fisheye.SimpleFishEyeEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 77 |
Method com.octo.captcha.engine.image.fisheye.SimpleFishEyeEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 80 |
Method com.octo.captcha.engine.image.fisheye.SimpleFishEyeEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 83 |
Method com.octo.captcha.engine.image.fisheye.SimpleFishEyeEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 86 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.engine.image.gimpy.BaffleListGimpyEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 49 |
Method com.octo.captcha.engine.image.gimpy.BaffleListGimpyEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 53 |
Method com.octo.captcha.engine.image.gimpy.BaffleListGimpyEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 56 |
Bug | Category | Details | Line |
---|---|---|---|
Dead store to emboss in com.octo.captcha.engine.image.gimpy.DeformedBaffleListGimpyEngine.buildInitialFactories() | STYLE | DLS_DEAD_LOCAL_STORE | 54 |
Method com.octo.captcha.engine.image.gimpy.DeformedBaffleListGimpyEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 110 |
Method com.octo.captcha.engine.image.gimpy.DeformedBaffleListGimpyEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 113 |
Method com.octo.captcha.engine.image.gimpy.DeformedBaffleListGimpyEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 116 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.engine.image.gimpy.NonLinearTextGimpyEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 47 |
Method com.octo.captcha.engine.image.gimpy.NonLinearTextGimpyEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 51 |
Method com.octo.captcha.engine.image.gimpy.NonLinearTextGimpyEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 54 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.engine.image.gimpy.SimpleListImageCaptchaEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 42 |
Method com.octo.captcha.engine.image.gimpy.SimpleListImageCaptchaEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 44 |
Method com.octo.captcha.engine.image.gimpy.SimpleListImageCaptchaEngine.buildInitialFactories() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 46 |
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in com.octo.captcha.engine.image.utils.ImageCaptchaToJPEG.main(String[]) | STYLE | REC_CATCH_EXCEPTION | 91 |
Exception is caught when Exception is not thrown in com.octo.captcha.engine.image.utils.ImageCaptchaToJPEG.main(String[]) | STYLE | REC_CATCH_EXCEPTION | 85 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.engine.image.utils.ImageToFile.serialize(BufferedImage, File) ignores return value of java.io.File.createNewFile() | CORRECTNESS | RV_RETURN_VALUE_IGNORED | 44 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.engine.image.utils.LogoGenerator.main(String[]) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 49 |
Method com.octo.captcha.engine.image.utils.LogoGenerator.main(String[]) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 51 |
Method com.octo.captcha.engine.image.utils.LogoGenerator.main(String[]) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 54 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.engine.sound.utils.SoundToFile.serialize(AudioInputStream, File) ignores return value of java.io.File.createNewFile() | CORRECTNESS | RV_RETURN_VALUE_IGNORED | 41 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.image.fisheye.FishEye is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 38-89 |
Bug | Category | Details | Line |
---|---|---|---|
Class com.octo.captcha.image.gimpy.Gimpy implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 25-55 |
com.octo.captcha.image.gimpy.Gimpy is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 25-55 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.image.gimpy.GimpyFactory.getRandomLength() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 98 |
Bug | Category | Details | Line |
---|---|---|---|
Redundant nullcheck of ?, which is known to be non-null in com.octo.captcha.module.config.CaptchaModuleConfig.validate() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 184 |
Bug | Category | Details | Line |
---|---|---|---|
Class com.octo.captcha.module.taglib.MessageTag implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 20-41 |
Bug | Category | Details | Line |
---|---|---|---|
Class com.octo.captcha.module.taglib.QuestionTag implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 17-28 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.service.AbstractManageableCaptchaService.generateCountTimeStampAndStoreCaptcha(String, Locale) invokes inefficient Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 338 |
Result of integer multiplication cast to long in com.octo.captcha.service.AbstractManageableCaptchaService.garbageCollectCaptchaStore(Iterator) | STYLE | ICAST_INTEGER_MULTIPLY_CAST_TO_LONG | 248 |
Result of integer multiplication cast to long in com.octo.captcha.service.AbstractManageableCaptchaService.getGarbageCollectableCaptchaIds(long) | STYLE | ICAST_INTEGER_MULTIPLY_CAST_TO_LONG | 289 |
Class com.octo.captcha.service.AbstractManageableCaptchaService implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 33-376 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.service.captchastore.CaptchaAndLocale is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 24-47 |
Bug | Category | Details | Line |
---|---|---|---|
Class com.octo.captcha.service.image.DefaultManageableImageCaptchaService implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 32-55 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.sound.gimpy.GimpySound is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 26-39 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.sound.gimpy.GimpySoundFactory.getRandomLength() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 98 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.sound.speller.SpellerSound is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 25-38 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.sound.speller.SpellerSoundFactory.getRandomLength() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 107 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.sound.spellfind.SpellFindCaptcha is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 25-38 |
Bug | Category | Details | Line |
---|---|---|---|
Method com.octo.captcha.sound.spellfind.SpellFindCaptchaFactory.getRandomLength() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 133 |
Method com.octo.captcha.sound.spellfind.SpellFindCaptchaFactory.getSoundCaptcha(Locale) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 97 |
Bug | Category | Details | Line |
---|---|---|---|
com.octo.captcha.text.math.MathCaptcha is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 22-43 |