[PR #201] [MERGED] Fix V3083 warnings from PVS-Studio Static Analyzer #1089

New issue

Closed

opened 2026-02-26 09:33:37 +03:00 by kerem · 0 comments

kerem commented 2026-02-26 09:33:37 +03:00

Owner

Copy link

📋 Pull Request Information

Original PR: https://github.com/NickeManarin/ScreenToGif/pull/201
Author: @spirifoxy
Created: 10/26/2017
Status: ✅ Merged
Merged: 10/27/2017
Merged by: @NickeManarin

Base: master ← Head: V3083_fix

---

📝 Commits (1)

• a59ba90 Fix V3083 warnings from PVS-Studio Static Analyzer

📊 Changes

6 files changed (+12 additions, -16 deletions)

View changed files

📝 GifRecorder/Util/UserActivityHook.cs (+4 -4)
📝 Other/HookTest/Util/UserActivityHook.cs (+4 -4)
📝 ScreenToGif/Controls/ImageScrollViewer.cs (+1 -2)
📝 ScreenToGif/Controls/SpectrumSlider.cs (+1 -4)
📝 ScreenToGif/Util/ActivityHook/UserActivityHook.cs (+1 -1)
📝 ScreenToGif/Webcam/DirectX/CaptureWebcam.cs (+1 -1)

📄 Description

I'm a member of the Pinguem.ru competition on finding errors in open source projects. A bug found using PVS-Studio. Warnings:

• V3083 Unsafe invocation of event 'ZoomChanged', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. ImageScrollViewer.cs 70
• V3083 Unsafe invocation of event 'AfterSelecting', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. SpectrumSlider.cs 82
• V3083 Unsafe invocation of event 'KeyUp', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 828
• V3083 Unsafe invocation of event 'CaptureFrameEvent', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. CaptureWebcam.cs 622
• V3083 Unsafe invocation of event 'OnMouseActivity', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 712
• V3083 Unsafe invocation of event 'KeyDown', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 760
• V3083 Unsafe invocation of event 'KeyPress', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 783
• V3083 Unsafe invocation of event 'KeyUp', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 796
• V3083 Unsafe invocation of event 'OnMouseActivity', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 768
• V3083 Unsafe invocation of event 'KeyDown', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 816
• V3083 Unsafe invocation of event 'KeyPress', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 841
• V3083 Unsafe invocation of event 'KeyUp', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 855

---

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/NickeManarin/ScreenToGif/pull/201 **Author:** [@spirifoxy](https://github.com/spirifoxy) **Created:** 10/26/2017 **Status:** ✅ Merged **Merged:** 10/27/2017 **Merged by:** [@NickeManarin](https://github.com/NickeManarin) **Base:** `master` ← **Head:** `V3083_fix` --- ### 📝 Commits (1) - [`a59ba90`](https://github.com/NickeManarin/ScreenToGif/commit/a59ba903b248ea23850db0bc615cee919be7b72b) Fix V3083 warnings from PVS-Studio Static Analyzer ### 📊 Changes **6 files changed** (+12 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `GifRecorder/Util/UserActivityHook.cs` (+4 -4) 📝 `Other/HookTest/Util/UserActivityHook.cs` (+4 -4) 📝 `ScreenToGif/Controls/ImageScrollViewer.cs` (+1 -2) 📝 `ScreenToGif/Controls/SpectrumSlider.cs` (+1 -4) 📝 `ScreenToGif/Util/ActivityHook/UserActivityHook.cs` (+1 -1) 📝 `ScreenToGif/Webcam/DirectX/CaptureWebcam.cs` (+1 -1) </details> ### 📄 Description I'm a member of the Pinguem.ru competition on finding errors in open source projects. A bug found using PVS-Studio. Warnings: - [V3083 ](https://www.viva64.com/en/w/v3083/)Unsafe invocation of event 'ZoomChanged', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. ImageScrollViewer.cs 70 - [V3083 ](https://www.viva64.com/en/w/v3083/)Unsafe invocation of event 'AfterSelecting', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. SpectrumSlider.cs 82 - [V3083 ](https://www.viva64.com/en/w/v3083/)Unsafe invocation of event 'KeyUp', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 828 - [V3083 ](https://www.viva64.com/en/w/v3083/)Unsafe invocation of event 'CaptureFrameEvent', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. CaptureWebcam.cs 622 - [V3083 ](https://www.viva64.com/en/w/v3083/)Unsafe invocation of event 'OnMouseActivity', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 712 - [V3083 ](https://www.viva64.com/en/w/v3083/)Unsafe invocation of event 'KeyDown', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 760 - [V3083 ](https://www.viva64.com/en/w/v3083/)Unsafe invocation of event 'KeyPress', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 783 - [V3083 ](https://www.viva64.com/en/w/v3083/)Unsafe invocation of event 'KeyUp', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 796 - [V3083 ](https://www.viva64.com/en/w/v3083/)Unsafe invocation of event 'OnMouseActivity', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 768 - [V3083 ](https://www.viva64.com/en/w/v3083/)Unsafe invocation of event 'KeyDown', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 816 - [V3083 ](https://www.viva64.com/en/w/v3083/)Unsafe invocation of event 'KeyPress', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 841 - [V3083 ](https://www.viva64.com/en/w/v3083/)Unsafe invocation of event 'KeyUp', NullReferenceException is possible. Consider assigning event to a local variable before invoking it. UserActivityHook.cs 855 --- _{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

kerem 2026-02-26 09:33:37 +03:00

• closed this issue
• added the
  pull-request
  label

kerem referenced this issue 2026-03-01 19:05:57 +03:00

[GH-ISSUE #1089] [Bug] ScreentoGif ARM64 won't start, throws exception #3026

kerem referenced this issue 2026-03-01 19:05:59 +03:00

[GH-ISSUE #1089] [Bug] ScreentoGif ARM64 won't start, throws exception #3029

kerem referenced this issue 2026-03-01 19:06:04 +03:00

[GH-ISSUE #1089] [Bug] ScreentoGif ARM64 won't start, throws exception #3032

Sign in to join this conversation.

No Branch/Tag specified

Branches Tags

master

dev

v2.40

2.43.1

2.43

2.42.1

2.42

2.41.5

2.41.4

2.41.3

2.41.2

2.41.1

2.41

2.40.1

2.40

2.39

2.38.1

2.38

2.37.2

2.37.1

2.37

2.36

2.35.4

2.35.3

2.35.2

2.35.1

2.35

2.34.1

2.34

2.33.1

2.33

2.32.1

2.32

2.31

2.30.1

2.30

2.29.1

2.29

2.28.2

2.28.1

2.28

2.27.3

2.27.2

2.27.1

2.27

2.26.1

2.26

2.25

2.24.2

2.24.1

2.24

2.23.2

2.23.1

2.23

2.22.1

2.22

2.21.2

2.21.1

2.21

2.20.4

2.20.3

2.20.2

2.20.1

2.20

2.19.3

2.19.2

2.19.1

2.19

2.18

2.17.1

2.17

2.16

2.15.2

2.15.1

2.15

2.14.1

2.14

2.13.3

2.13.2

2.13.1

2.13

2.12.1

2.12

2.11

2.10

2.9

2.8.1

2.8

2.7.3

2.7.2

2.7.1

2.7

2.6

2.5

2.4

2.3.2

2.3.1

2.3

2.2

2.1

Labels

Clear labels   

copy cats

  

duplicated

  

future feature

  

pull-request

Mirrored from GitHub Pull Request

  

⬜ Accepted

  

⬜ Completed

  

⬜ Help Wanted 💪

  

⬜ In Progress

  

⬜ Missing Details

  

⬜ Pending

  

⬜ Waiting For Answer ⏳

  

🆕 feature preview

  

🔷 Bug 🐛

  

🔷 Out Of Scope

  

🔷 Out Of Scope

  

🔷 Question

  

🔷Enhancement

  

🔷Enhancement

  

🔷Invalid / External

  

🔷Knowledge Base

  

🔷Won't Fix

  

🕑 High

  

🕑 High

  

🕑 High

  

🕕 Medium

  

🕙 Low

  

🕛 Critical

No labels

copy cats

duplicated

future feature

pull-request

⬜ Accepted

⬜ Completed

⬜ Help Wanted 💪

⬜ In Progress

⬜ Missing Details

⬜ Pending

⬜ Waiting For Answer ⏳

🆕 feature preview

🔷 Bug 🐛

🔷 Out Of Scope

🔷 Out Of Scope

🔷 Question

🔷Enhancement

🔷Enhancement

🔷Invalid / External

🔷Knowledge Base

🔷Won't Fix

🕑 High

🕑 High

🕑 High

🕕 Medium

🕙 Low

🕛 Critical

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/ScreenToGif#1089

No description provided.