Requête + code Java

Version imprimable

Requête + code Java

Bonjour,
Prière de m'aider

Mon but dans le code (Servlet) qui suit c'est de filtrer la base de données selon un formulaire (jsp)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
package aaa;
 
import java.io.IOException;
import java.io.PrintWriter;
import java.sql.SQLException;
import java.util.Enumeration;
 
import javax.servlet.ServletException;
import javax.servlet.annotation.WebServlet;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
 
import bdd.connexion;
 
/**
 * Servlet implementation class trtm
 */
@WebServlet("/trtm")
public class trtm extends HttpServlet {
	private static final long serialVersionUID = 1L;
 
    /**
     * @see HttpServlet#HttpServlet()
     */
    public trtm() {
        super();
        // TODO Auto-generated constructor stub
    }
 
	/**
	 * @see HttpServlet#doGet(HttpServletRequest request, HttpServletResponse response)
	 */
	protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		// TODO Auto-generated method stub
		doPost(request, response);
	}
 
	/**
	 * @see HttpServlet#doPost(HttpServletRequest request, HttpServletResponse response)
	 */
	protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		// TODO Auto-generated method stub
 
		PrintWriter out = response.getWriter();
		connexion cn = new connexion("inwi");
		connexion cn1 = new connexion("inwi");
 
		String requete = "select * from inwi1 where ";
		String time = request.getParameter("timestamp");
        String[] valeurs = request.getParameterValues("NEType");
 
		Enumeration<String> Noms = request.getParameterNames();
		String sss="";
		while (Noms.hasMoreElements()) {
			String string = (String) Noms.nextElement();
			sss=sss+"<hr>"+string+"</hr>";
 
		}
        out.println(sss);
 
		if (request.getParameterValues("NEType") != null){
 
			requete = requete+"NEType IN ('";
			valeurs = request.getParameterValues("NEType");
			for (int i = 0; i < valeurs.length; i++) {
 
				requete=requete+valeurs[i];
				if (i==valeurs.length-1) {
					requete=requete+"')";
				}
				else requete=requete+"','";
			}
 
		}
		out.println(requete);
 
		if (request.getParameterValues("NEName") != null) {
			valeurs = request.getParameterValues("NEName");
			requete=requete+"AND NEName IN ('";
			for (int i = 0; i < valeurs.length; i++) {
 
				requete=requete+valeurs[i];
				if (i==valeurs.length-1) {
					requete=requete+"')";
				}
				else requete=requete+"','";
			}
 
		}
		out.println(requete);
 
		if (request.getParameterValues("NEIP") != null) {
			valeurs = request.getParameterValues("NEIP");
			requete = requete+"AND NEIP IN ('";
			for (int i = 0; i < valeurs.length; i++) {
 
				requete=requete+valeurs[i];
				if (i==valeurs.length-1) {
					requete=requete+"')";
				}
				else requete=requete+"','";
			}
		}
		out.println(requete);
 
 
		if (request.getParameterValues("Operator") != null) {
			valeurs = request.getParameterValues("Operator");
			requete = requete+"AND Operator IN ('";
			for (int i = 0; i < valeurs.length; i++) {
 
				requete=requete+valeurs[i];
				if (i==valeurs.length-1) {
					requete=requete+"')";
				}
				else requete=requete+"','";
			}
		}
		out.println(requete);
 
 
		if (request.getParameterValues("Terminal") != null) {
			valeurs = request.getParameterValues("Terminal");
			requete =requete+ "AND Terminal IN ('";
			for (int i = 0; i < valeurs.length; i++) {
 
				requete=requete+valeurs[i];
				if (i==valeurs.length-1) {
					requete=requete+"')";
				}
				else requete=requete+"','";
			}
		}
		out.println(requete);
		////////////////////////////////////////////////////////////////////////*/*/*
		///////////////////////////////////////////////////////////////////////
		//////////////////////////////////////////////////////////////////////
		if (request.getParameter("timestamp") != null) {
			time = request.getParameter("timestamp");
			requete =requete+ "AND MMLStartTime like '%"+time+"%'  ";
 
		}
		out.println(requete);
 
 
		if (request.getParameterValues("result") != null) {
			valeurs = request.getParameterValues("result");
			requete =requete+ "AND result IN ('";
			for (int i = 0; i < valeurs.length; i++) {
 
				requete=requete+valeurs[i];
				if (i==(valeurs.length-1)) {
					requete=requete+"')";
				}
				else requete=requete+"','";
			}
		}
		out.println(requete);
 
		if (request.getParameterValues("Domain") != null) {
			valeurs = request.getParameterValues("Domain");
			requete =requete+ "AND Domain IN ('";
			for (int i = 0; i < valeurs.length; i++) {
 
				requete=requete+valeurs[i];
				if (i==valeurs.length-1) {
					requete=requete+"')";
				}
				else requete=requete+"','";
			}
		}
		out.println(requete);
 
 
		if (request.getParameterValues("MMLcmd") != null) {
			valeurs = request.getParameterValues("MMLcmd");
			requete =requete+ "AND MML REGEXP '"; 
			for (int i = 0; i < valeurs.length; i++) {
 
				requete=requete+valeurs[i];
				if (i==(valeurs.length-1)) {
					requete=requete+" ' ";
				}
				else requete=requete+"|";
			}
		}
		out.println(requete);
 
		//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
		cn.Lire(requete);
 
		try {
			while (cn.getrs().next()) {
				String NEType = null; String NEName= null; String NEIP= null;
				String Operator= null; String Terminal= null; String MMLStartTime= null;
				String MMLEndTime= null; String result= null; String MML = null; String Message= null;String Domain= null;
				try {
					 NEType = cn.getrs().getString(1);
					 NEName = cn.getrs().getString(2);
					 NEIP = cn.getrs().getString(3);
					 Operator = cn.getrs().getString(4);
					 Terminal = cn.getrs().getString(5);
					 MMLStartTime = cn.getrs().getString(6);
					 MMLEndTime = cn.getrs().getString(7);
					 result = cn.getrs().getString(8);
					 MML = cn.getrs().getString(9);
					 Message = cn.getrs().getString(10);
					 Domain = cn.getrs().getString(11);
				} catch (Exception e) {
				}
 
				String req = "insert into `template`(TemplateName,AccesRight,NEType,NEName,NEIP,Operator,Terminal,MMLStartTime,MMLEndTime,result,MML,Message,Domain) values ('"
						+ request.getParameter("name")
						+ "','"
						+ request.getParameter("right")
						+ "','"
						+ NEType
						+ "','"
						+ NEName
						+ "','"
						+ NEIP
						+ "','"
						+ Operator
						+ "','"
						+ Terminal
						+ "','"
						+ MMLStartTime
						+ "','"
						+ MMLEndTime
						+ "','"
						+ result
						+ "','" + MML + "','" + Message + "','" + Domain + "')";
 
				cn1.MiseAJour(req);
			}
		} catch (SQLException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
 
		request.getRequestDispatcher("ManuelResult.jsp?cond=ok").forward(request, response);
		{
 
		} 
 
	}
 
 
}

le filtrage se fait correctement au niveau de tout les champs sauf celui du MML est exactement cette partie du code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
if (request.getParameterValues("MMLcmd") != null) {
			valeurs = request.getParameterValues("MMLcmd");
			requete =requete+ "AND MML REGEXP '"; 
			for (int i = 0; i < valeurs.length; i++) {
 
				requete=requete+valeurs[i];
				if (i==(valeurs.length-1)) {
					requete=requete+" ' ";
				}
				else requete=requete+"|";
			}
		}
		out.println(requete);

J'aimerais bien savoir l'erreur que j'ai fait si pouvez m'aider :cry:
Merci d'avance

salut,

je demande encore votre aide:calim2:

je me suis retourner vers LIKE mais ça ne fonctionne pas voilà le code avec LIKE
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 if (request.getParameterValues("MMLcmd") != null) { valeurs = request.getParameterValues("MMLcmd"); requete =requete+ "AND ( MML LIKE '% "; for (int i = 0; i < valeurs.length; i++) { requete=requete+valeurs[i]; if (i==(valeurs.length-1)) { requete=requete+" %')"; } else requete=requete+" OR MML LIKE '% "; } } out.println(requete);
SVP aidez moi à trouver l'erreur :cry:

salut

tu peux poster ce que donne le system.out de ta requête ?
Comme ça, je dirais que tu ne fermes pas les quotes du like

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
// Attention, ajouter un test de non vide
if (request.getParameterValues("MMLcmd") != null) {     
     valeurs = request.getParameterValues("MMLcmd");     
     requete = requete+ " AND ( ";      
 
     for (int i = 0; i < valeurs.length; i++) {
          requete=requete+ "MML LIKE '%" + valeurs[i] + "%'";
          if (i<valeurs.length-1) {
             requete=requete + " OR ";
         }
     }
    requete = requete + ")";
} 
out.println(requete);

ça me paraît un peu mieux comme ça :)

25/07/2012, 16h05
tchize_

1) tes requêtes sont incomplètes
2) t'es occupé d'ouvrir à tes utilisateurs une voie royale vers l'attaque par injection SQL. Pour résoudre ces deux cas (rendre les requetes visible et sécurisée), utilise des prepared statements.
25/07/2012, 17h13
sihamnet

Merci infiniment florentB :merci:

Citation:

Envoyé par tchize_

1) tes requêtes sont incomplètes
2) t'es occupé d'ouvrir à tes utilisateurs une voie royale vers l'attaque par injection SQL. Pour résoudre ces deux cas (rendre les requetes visible et sécurisée), utilise des prepared statements.

Salut,
en effet j'utilise une classe connexion voilà son code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
import java.sql.*;
public class connexion {
 
 
   private Connection con;
   private Statement st;
   protected ResultSet rs;
   String DB="inwi";
 
      //Constructeur
     public connexion(String DB) {  
    	 String url="jdbc:mysql://localhost/"+DB;                         //Définition de l’URL de connexion
         try {
                   Class.forName("com.mysql.jdbc.Driver");                //Chargement du pilote JDBC
                   con = DriverManager.getConnection(url, "root", "");    //Etablissement de la connexion
                   st = con.createStatement();                           /*Afin d’accéder ou de modifier les informations contenues dans la base de données, 
                                                                          il convient d’utiliser un objet de type Statement. 
                                                                          Une instance de cet objet est retournée par cette méthode*/ 
         }  
        catch (ClassNotFoundException e) {
         System.err.println("Problème de pilote");
        }
        catch (SQLException c) {
         System.err.println("Base de données non trouvée ou requête incorrecte");
        }
   }
 
   public void Lire(String req) {                         //Selectionner la Base de Données
	   try {
		   rs=st.executeQuery(req);
		   } catch (SQLException e) {
		   	// TODO Auto-generated catch block
		   	e.printStackTrace(); 
		    }
   }
 
   public void MiseAJour(String req) {                   //Modifier la Base de Données
	   try {
			st.executeUpdate(req);
		} catch (SQLException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
   }
 
   public void insert(String NEType, String NEName, String NEIP,
			String Operator, String Terminal, String MMLStartTime,
			String MMLEndTime, String result, String MML, String Message,
			String Domain) {
 
		try {
			st.executeUpdate("insert into inwi1(NEType,NEName,NEIP,Operator,Terminal,MMLStartTime,MMLEndTime,result,MML,Message,Domain) values ('"+NEType+"','"+NEName+"','"+NEIP+"','"+Operator+"','"+Terminal+"','"+MMLStartTime+"','"+MMLEndTime+"','"+result+"','"+MML+"','"+Message+"','"+Domain+"')");
		} catch (SQLException e) {
			e.printStackTrace();
		}
	}
 
   public boolean Suiv() {
	   try {
			return rs.next();
		} catch (SQLException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
			return false;
		}
   }
 
   public void fermer(){                                 // Fermer le connexion
       try {
            con.close();
        } catch (SQLException e) {
           // TODO Auto-generated catch block
           e.printStackTrace();
         }
   }
 
   public ResultSet getrs(){
       return rs;	
   }
 
}

est ce que c'est bon ou je dois améliorer quelque chose :roll:
et merci d'avance :mrgreen:

25/07/2012, 22h19
tchize_

:faq: Qu'est-ce qu'un PreparedStatement ?
D'une manière générale, c'est une très mauvaise idée de concaténer des paramètres dans une requête SQL.