More CodeQL fixes

This commit is contained in:
Kovid Goyal
2025-04-20 22:13:45 +05:30
parent 237bb35ee9
commit 341df0dccb
7 changed files with 20 additions and 22 deletions

View File

@@ -81,8 +81,7 @@ func (self *if_panel) on_click(id string) (err error) {
if scheme != "fval" {
return
}
v, _ := strconv.ParseUint(val, 10, 64)
v, _ := strconv.ParseUint(val, 10, 0)
if err = self.handler.face_pane.change_feature_value(self.feat_tag, uint(v), false); err != nil {
return err
}
@@ -143,7 +142,7 @@ func (self *if_panel) on_enter(family, which string, settings faces_settings, fe
self.current_val = current_val
self.rl.ResetText()
if self.current_val > 0 {
self.rl.SetText(strconv.Itoa(int(self.current_val)))
self.rl.SetText(strconv.FormatUint(uint64(self.current_val), 10))
}
return self.handler.draw_screen()
}

View File

@@ -165,20 +165,20 @@ func main(cmd *cli.Command, o *Options, args []string) (rc int, err error) {
return 1, fmt.Errorf("Invalid size specification: " + opts.UseWindowSize)
}
screen_size = &unix.Winsize{}
t := 0
if t, err = strconv.Atoi(parts[0]); err != nil || t < 1 {
var t uint64
if t, err = strconv.ParseUint(parts[0], 10, 16); err != nil || t < 1 {
return 1, fmt.Errorf("Invalid size specification: %s with error: %w", opts.UseWindowSize, err)
}
screen_size.Col = uint16(t)
if t, err = strconv.Atoi(parts[1]); err != nil || t < 1 {
if t, err = strconv.ParseUint(parts[1], 10, 16); err != nil || t < 1 {
return 1, fmt.Errorf("Invalid size specification: %s with error: %w", opts.UseWindowSize, err)
}
screen_size.Row = uint16(t)
if t, err = strconv.Atoi(parts[2]); err != nil || t < 1 {
if t, err = strconv.ParseUint(parts[2], 10, 16); err != nil || t < 1 {
return 1, fmt.Errorf("Invalid size specification: %s with error: %w", opts.UseWindowSize, err)
}
screen_size.Xpixel = uint16(t)
if t, err = strconv.Atoi(parts[3]); err != nil || t < 1 {
if t, err = strconv.ParseUint(parts[3], 10, 16); err != nil || t < 1 {
return 1, fmt.Errorf("Invalid size specification: %s with error: %w", opts.UseWindowSize, err)
}
screen_size.Ypixel = uint16(t)

View File

@@ -14,6 +14,7 @@ import (
"strconv"
"strings"
"unicode"
"unicode/utf8"
"kitty/tools/cli"
"kitty/tools/tty"
@@ -71,8 +72,8 @@ func parse_favorites(raw string) (ans []rune) {
line = line[:idx]
}
code_text, _, _ := strings.Cut(line, " ")
code, err := strconv.ParseUint(code_text, 16, 32)
if err == nil && codepoint_ok(rune(code)) {
code, err := strconv.ParseInt(code_text, 16, 32)
if err == nil && code <= utf8.MaxRune && codepoint_ok(rune(code)) {
ans = append(ans, rune(code))
}
}
@@ -209,7 +210,6 @@ func is_index(word string) bool {
}
func (self *handler) update_codepoints() {
var index_word uint64
var q checkpoints_key
q.mode = self.mode
q.index_word = -1
@@ -233,8 +233,9 @@ func (self *handler) update_codepoints() {
if i > 0 && is_index(w) {
iw := words[i]
words = words[:i]
index_word, _ = strconv.ParseUint(strings.TrimLeft(iw, INDEX_CHAR), INDEX_BASE, 32)
q.index_word = int(index_word)
if index_word, perr := strconv.ParseInt(strings.TrimLeft(iw, INDEX_CHAR), INDEX_BASE, 0); perr == nil {
q.index_word = int(index_word)
}
break
}
}

View File

@@ -31,8 +31,8 @@ func resolved_char(ch rune, emoji_variation string) string {
}
func decode_hint(text string) int {
x, err := strconv.ParseUint(text, INDEX_BASE, 32)
if err != nil {
x, err := strconv.ParseInt(text, INDEX_BASE, 0)
if err != nil || x < 0 {
return -1
}
return int(x)