Skip to content

Commit d571cda

Browse files
committed
unmask AffectedRows
1 parent 80ad073 commit d571cda

File tree

2 files changed

+18
-32
lines changed

2 files changed

+18
-32
lines changed

connection.go

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -158,22 +158,13 @@ func (mc *mysqlConn) exec(query string) (err error) {
158158
// Read Result
159159
var resLen int
160160
resLen, err = mc.readResultSetHeaderPacket()
161-
if err != nil {
162-
return
163-
}
164-
165-
if resLen > 0 {
161+
if err == nil && resLen > 0 {
166162
_, err = mc.readUntilEOF()
167163
if err != nil {
168164
return
169165
}
170166

171-
if mc.affectedRows > 0 {
172-
_, err = mc.readUntilEOF()
173-
return
174-
}
175-
176-
mc.affectedRows, err = mc.readUntilEOF()
167+
_, err = mc.readUntilEOF()
177168
}
178169

179170
return

statement.go

Lines changed: 16 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -43,31 +43,26 @@ func (stmt *mysqlStmt) Exec(args []driver.Value) (driver.Result, error) {
4343
// Read Result
4444
var resLen int
4545
resLen, err = stmt.mc.readResultSetHeaderPacket()
46-
if err != nil {
47-
return nil, err
48-
}
49-
50-
if resLen > 0 {
51-
// Columns
52-
_, err = stmt.mc.readUntilEOF()
53-
if err != nil {
54-
return nil, err
46+
if err == nil {
47+
if resLen > 0 {
48+
// Columns
49+
_, err = stmt.mc.readUntilEOF()
50+
if err != nil {
51+
return nil, err
52+
}
53+
54+
// Rows
55+
_, err = stmt.mc.readUntilEOF()
5556
}
56-
57-
// Rows
58-
stmt.mc.affectedRows, err = stmt.mc.readUntilEOF()
59-
if err != nil {
60-
return nil, err
57+
if err == nil {
58+
return &mysqlResult{
59+
affectedRows: int64(stmt.mc.affectedRows),
60+
insertId: int64(stmt.mc.insertId),
61+
}, nil
6162
}
6263
}
63-
if err != nil {
64-
return nil, err
65-
}
6664

67-
return &mysqlResult{
68-
affectedRows: int64(stmt.mc.affectedRows),
69-
insertId: int64(stmt.mc.insertId),
70-
}, nil
65+
return nil, err
7166
}
7267

7368
func (stmt *mysqlStmt) Query(args []driver.Value) (driver.Rows, error) {

0 commit comments

Comments
 (0)