Skip to content
This repository was archived by the owner on Jan 5, 2023. It is now read-only.

Commit df71f0b

Browse files
committed
Remove ReadByte, WriteByte, ReadRune, WriteRune
1 parent ff81ad6 commit df71f0b

File tree

4 files changed

+102
-256
lines changed

4 files changed

+102
-256
lines changed

ql/src/semmle/go/frameworks/stdlib/Bufio.qll

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -70,10 +70,6 @@ module Bufio {
7070
this.hasQualifiedName("bufio", "Reader", "Peek") and
7171
(inp.isReceiver() and outp.isResult(0))
7272
or
73-
// signature: func (*Reader).ReadByte() (byte, error)
74-
this.hasQualifiedName("bufio", "Reader", "ReadByte") and
75-
(inp.isReceiver() and outp.isResult(0))
76-
or
7773
// signature: func (*Reader).ReadBytes(delim byte) ([]byte, error)
7874
this.hasQualifiedName("bufio", "Reader", "ReadBytes") and
7975
(inp.isReceiver() and outp.isResult(0))
@@ -82,10 +78,6 @@ module Bufio {
8278
this.hasQualifiedName("bufio", "Reader", "ReadLine") and
8379
(inp.isReceiver() and outp.isResult(0))
8480
or
85-
// signature: func (*Reader).ReadRune() (r rune, size int, err error)
86-
this.hasQualifiedName("bufio", "Reader", "ReadRune") and
87-
(inp.isReceiver() and outp.isResult(0))
88-
or
8981
// signature: func (*Reader).ReadSlice(delim byte) (line []byte, err error)
9082
this.hasQualifiedName("bufio", "Reader", "ReadSlice") and
9183
(inp.isReceiver() and outp.isResult(0))
@@ -122,14 +114,6 @@ module Bufio {
122114
this.hasQualifiedName("bufio", "Writer", "Write") and
123115
(inp.isParameter(0) and outp.isReceiver())
124116
or
125-
// signature: func (*Writer).WriteByte(c byte) error
126-
this.hasQualifiedName("bufio", "Writer", "WriteByte") and
127-
(inp.isParameter(0) and outp.isReceiver())
128-
or
129-
// signature: func (*Writer).WriteRune(r rune) (size int, err error)
130-
this.hasQualifiedName("bufio", "Writer", "WriteRune") and
131-
(inp.isParameter(0) and outp.isReceiver())
132-
or
133117
// signature: func (*Writer).WriteString(s string) (int, error)
134118
this.hasQualifiedName("bufio", "Writer", "WriteString") and
135119
(inp.isParameter(0) and outp.isReceiver())

ql/src/semmle/go/frameworks/stdlib/Bytes.qll

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -158,10 +158,6 @@ module Bytes {
158158
this.hasQualifiedName("bytes", "Buffer", "Next") and
159159
(inp.isReceiver() and outp.isResult())
160160
or
161-
// signature: func (*Buffer).ReadByte() (byte, error)
162-
this.hasQualifiedName("bytes", "Buffer", "ReadByte") and
163-
(inp.isReceiver() and outp.isResult(0))
164-
or
165161
// signature: func (*Buffer).ReadBytes(delim byte) (line []byte, err error)
166162
this.hasQualifiedName("bytes", "Buffer", "ReadBytes") and
167163
(inp.isReceiver() and outp.isResult(0))
@@ -170,10 +166,6 @@ module Bytes {
170166
this.hasQualifiedName("bytes", "Buffer", "ReadFrom") and
171167
(inp.isParameter(0) and outp.isReceiver())
172168
or
173-
// signature: func (*Buffer).ReadRune() (r rune, size int, err error)
174-
this.hasQualifiedName("bytes", "Buffer", "ReadRune") and
175-
(inp.isReceiver() and outp.isResult(0))
176-
or
177169
// signature: func (*Buffer).ReadString(delim byte) (line string, err error)
178170
this.hasQualifiedName("bytes", "Buffer", "ReadString") and
179171
(inp.isReceiver() and outp.isResult(0))
@@ -186,14 +178,6 @@ module Bytes {
186178
this.hasQualifiedName("bytes", "Buffer", "Write") and
187179
(inp.isParameter(0) and outp.isReceiver())
188180
or
189-
// signature: func (*Buffer).WriteByte(c byte) error
190-
this.hasQualifiedName("bytes", "Buffer", "WriteByte") and
191-
(inp.isParameter(0) and outp.isReceiver())
192-
or
193-
// signature: func (*Buffer).WriteRune(r rune) (n int, err error)
194-
this.hasQualifiedName("bytes", "Buffer", "WriteRune") and
195-
(inp.isParameter(0) and outp.isReceiver())
196-
or
197181
// signature: func (*Buffer).WriteString(s string) (n int, err error)
198182
this.hasQualifiedName("bytes", "Buffer", "WriteString") and
199183
(inp.isParameter(0) and outp.isReceiver())
@@ -206,14 +190,6 @@ module Bytes {
206190
this.hasQualifiedName("bytes", "Reader", "ReadAt") and
207191
(inp.isReceiver() and outp.isParameter(0))
208192
or
209-
// signature: func (*Reader).ReadByte() (byte, error)
210-
this.hasQualifiedName("bytes", "Reader", "ReadByte") and
211-
(inp.isReceiver() and outp.isResult(0))
212-
or
213-
// signature: func (*Reader).ReadRune() (ch rune, size int, err error)
214-
this.hasQualifiedName("bytes", "Reader", "ReadRune") and
215-
(inp.isReceiver() and outp.isResult(0))
216-
or
217193
// signature: func (*Reader).Reset(b []byte)
218194
this.hasQualifiedName("bytes", "Reader", "Reset") and
219195
(inp.isParameter(0) and outp.isReceiver())

ql/test/library-tests/semmle/go/frameworks/StdlibTaintFlow/Bufio.go

Lines changed: 51 additions & 97 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)